{"action":"created","issue":{"url":"https://api.github.com/repos/openssl/openssl/issues/1628","repository_url":"https://api.github.com/repos/openssl/openssl","labels_url":"https://api.github.com/repos/openssl/openssl/issues/1628/labels{/name}","comments_url":"https://api.github.com/repos/openssl/openssl/issues/1628/comments","events_url":"https://api.github.com/repos/openssl/openssl/issues/1628/events","html_url":"https://github.com/openssl/openssl/pull/1628","id":179464457,"number":1628,"title":"Support building with cygwin perl","user":{"login":"yoe","id":42075,"avatar_url":"https://avatars.githubusercontent.com/u/42075?v=3","gravatar_id":"","url":"https://api.github.com/users/yoe","html_url":"https://github.com/yoe","followers_url":"https://api.github.com/users/yoe/followers","following_url":"https://api.github.com/users/yoe/following{/other_user}","gists_url":"https://api.github.com/users/yoe/gists{/gist_id}","starred_url":"https://api.github.com/users/yoe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yoe/subscriptions","organizations_url":"https://api.github.com/users/yoe/orgs","repos_url":"https://api.github.com/users/yoe/repos","events_url":"https://api.github.com/users/yoe/events{/privacy}","received_events_url":"https://api.github.com/users/yoe/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-09-27T11:14:14Z","updated_at":"2016-09-27T12:40:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openssl/openssl/pulls/1628","html_url":"https://github.com/openssl/openssl/pull/1628","diff_url":"https://github.com/openssl/openssl/pull/1628.diff","patch_url":"https://github.com/openssl/openssl/pull/1628.patch"},"body":"The match on the line just above the open, when using cygwin perl, does\r\nnot end with a / character. The result of that is that we try to find a\r\nfile \"ms../crypto/perlasm/x86_64-xlate.pl\", which obviously does not\r\nexist.\r\n\r\nFix by adding an explicit / character\r\n\r\nSigned-off-by: Wouter Verhelst <w@uter.be>"},"comment":{"url":"https://api.github.com/repos/openssl/openssl/issues/comments/249852812","html_url":"https://github.com/openssl/openssl/pull/1628#issuecomment-249852812","issue_url":"https://api.github.com/repos/openssl/openssl/issues/1628","id":249852812,"user":{"login":"levitte","id":698918,"avatar_url":"https://avatars.githubusercontent.com/u/698918?v=3","gravatar_id":"","url":"https://api.github.com/users/levitte","html_url":"https://github.com/levitte","followers_url":"https://api.github.com/users/levitte/followers","following_url":"https://api.github.com/users/levitte/following{/other_user}","gists_url":"https://api.github.com/users/levitte/gists{/gist_id}","starred_url":"https://api.github.com/users/levitte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/levitte/subscriptions","organizations_url":"https://api.github.com/users/levitte/orgs","repos_url":"https://api.github.com/users/levitte/repos","events_url":"https://api.github.com/users/levitte/events{/privacy}","received_events_url":"https://api.github.com/users/levitte/received_events","type":"User","site_admin":false},"created_at":"2016-09-27T12:40:29Z","updated_at":"2016-09-27T12:40:29Z","body":"This is cygwin perl used from the Windows Command Prompt or similar, right?  What happens is that `${dir}` gets the value `ms\\\\`, and the command line string that's being opened becomes `| \"/usr/bin/perl\" ms\\\\../crypto/perlasm/x86_64-xlate.pl`, which is kinda sorta correct, except that Cygwin perl will massage `\\\\` in a way that doesn't work well in the end, resulting in the attempt to load ms../crypto/perlasm/x86_64-xlate.pl...\r\n\r\nYour fix comes with a different problem, though, which you will see if you do this:\r\n\r\n```\r\ncd ms\r\nperl uplink-x86_64.pl\r\n```\r\n\r\nthen, it will suddenly try to load `/../crypto/perlasm/x86_64-xlate.pl`, and that's not what you intended.\r\n\r\nBecause this and other quirks we've found along the way, we *do not* support using cygwin perl in native Windows command line, just like we don't support the use of  ActiveState Perl in cygwin.\r\n\r\nHowever, if you still want to fix this particular issue properly, I'd say adding this line rather than your slash has a greater chance of achieving something:\r\n\r\n``` perl\r\n$dir =~ s|\\\\|/|g;\r\n```\r\n\r\nTry it out."}}
{"actor":{"display_login":"levitte"}}