{"action":"created","issue":{"url":"https://api.github.com/repos/openssl/openssl/issues/5310","repository_url":"https://api.github.com/repos/openssl/openssl","labels_url":"https://api.github.com/repos/openssl/openssl/issues/5310/labels{/name}","comments_url":"https://api.github.com/repos/openssl/openssl/issues/5310/comments","events_url":"https://api.github.com/repos/openssl/openssl/issues/5310/events","html_url":"https://github.com/openssl/openssl/issues/5310","id":296042761,"number":5310,"title":"Inconsistent generation logic for asm modules","user":{"login":"vdukhovni","id":1107986,"avatar_url":"https://avatars2.githubusercontent.com/u/1107986?v=4","gravatar_id":"","url":"https://api.github.com/users/vdukhovni","html_url":"https://github.com/vdukhovni","followers_url":"https://api.github.com/users/vdukhovni/followers","following_url":"https://api.github.com/users/vdukhovni/following{/other_user}","gists_url":"https://api.github.com/users/vdukhovni/gists{/gist_id}","starred_url":"https://api.github.com/users/vdukhovni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vdukhovni/subscriptions","organizations_url":"https://api.github.com/users/vdukhovni/orgs","repos_url":"https://api.github.com/users/vdukhovni/repos","events_url":"https://api.github.com/users/vdukhovni/events{/privacy}","received_events_url":"https://api.github.com/users/vdukhovni/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":11,"created_at":"2018-02-09T23:22:43Z","updated_at":"2018-02-11T18:06:08Z","closed_at":null,"author_association":"NONE","body":"In commits 10eab320a5e4b585f640b5f11d21eebaebb1814a, 184bc45f683c76531d7e065b6553ca9086564576 and eb77e8886df84526f42f566632be71d4ed373308  and perhaps other commits most of the assembly modules were modified to take an output file name.  But apparently not all.  NetBSD does cross-builds and needs to generate the Perl asm files directly bypassing Configure.  It would be good to have the logic be consistent across all architectures.\r\n\r\nStill writing to STDOUT:\r\n```\r\ncrypto/des/asm/desboth.pl\r\ncrypto/rc4/asm/rc4-c64xplus.pl\r\ncrypto/sha/asm/keccak1600-armv4.pl\r\ncrypto/sha/asm/keccak1600-avx2.pl\r\ncrypto/sha/asm/keccak1600-avx512.pl\r\ncrypto/sha/asm/keccak1600-avx512vl.pl\r\ncrypto/sha/asm/keccak1600-c64x.pl\r\n```"},"comment":{"url":"https://api.github.com/repos/openssl/openssl/issues/comments/364772870","html_url":"https://github.com/openssl/openssl/issues/5310#issuecomment-364772870","issue_url":"https://api.github.com/repos/openssl/openssl/issues/5310","id":364772870,"user":{"login":"vdukhovni","id":1107986,"avatar_url":"https://avatars2.githubusercontent.com/u/1107986?v=4","gravatar_id":"","url":"https://api.github.com/users/vdukhovni","html_url":"https://github.com/vdukhovni","followers_url":"https://api.github.com/users/vdukhovni/followers","following_url":"https://api.github.com/users/vdukhovni/following{/other_user}","gists_url":"https://api.github.com/users/vdukhovni/gists{/gist_id}","starred_url":"https://api.github.com/users/vdukhovni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vdukhovni/subscriptions","organizations_url":"https://api.github.com/users/vdukhovni/orgs","repos_url":"https://api.github.com/users/vdukhovni/repos","events_url":"https://api.github.com/users/vdukhovni/events{/privacy}","received_events_url":"https://api.github.com/users/vdukhovni/received_events","type":"User","site_admin":false},"created_at":"2018-02-11T18:06:08Z","updated_at":"2018-02-11T18:06:08Z","author_association":"NONE","body":"> @richsalz crypto/des/asm/desboth.pl is required by crypto/des/asm/des-586.pl, so no worry there.\r\n\r\nIt would also help if modules that were only `required` by other modules and  do not directly produce asm code were called `something.pm` rather than `something.pl`.  I know it is not something we need, but I think that it is good to make it easier for downstream distributions to bundle OpenSSL (rather than consider alternatives). So I am not calling out *bugs*, rather asking for I think are reasonable accommodations, that perhaps are also likely to make our own code somewhat easier to work with."}}
{"actor":{"display_login":"vdukhovni"}}