Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Link to CRYPTOGAMS license is no longer accessible #23499

Open
ueno opened this issue Feb 7, 2024 · 1 comment · May be fixed by #24231
Open

Link to CRYPTOGAMS license is no longer accessible #23499

ueno opened this issue Feb 7, 2024 · 1 comment · May be fixed by #24231
Labels
branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 branch: 3.2 Merge to openssl-3.2 triaged: documentation The issue/pr deals with documentation (errors)
Milestone
3.4.0

Comments

@ueno
Copy link
Contributor

ueno commented Feb 7, 2024 •
edited

Some of the Perl scripts generating assembly (e.g., crypto/modes/asm/aesni-gcm-x86_64.pl) have this notice:

# Written by Andy Polyakov <appro@openssl.org> for the OpenSSL
# project. The module is, however, dual licensed under OpenSSL and
# CRYPTOGAMS licenses depending on where you obtain it. For further
# details see http://www.openssl.org/~appro/cryptogams/.

However, the above URL is not accessible anymore. It would be nice if the URL is up again (or the link is changed to the repository), as well as the license file is included in the OpenSSL distribution.
@ueno ueno added the issue: documentation The issue reports errors in (or missing) documentation label Feb 7, 2024
@mattcaswell mattcaswell added triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly and removed issue: documentation The issue reports errors in (or missing) documentation labels Feb 7, 2024
@t8m t8m added branch: master Merge to master branch triaged: documentation The issue/pr deals with documentation (errors) branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 branch: 3.2 Merge to openssl-3.2 and removed triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly labels Feb 7, 2024
@Max-Deng-203 Max-Deng-203 linked a pull request Apr 22, 2024 that will close this issue
Open
2 tasks
@nhorman
Copy link
Contributor

nhorman commented Apr 29, 2024

Proposal:
302/redirect broken link to github LICENSE file
update code to point to github tree
Ask andy if attribute fixup is ok

@nhorman nhorman added this to the 3.4.0 milestone Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 branch: 3.2 Merge to openssl-3.2 triaged: documentation The issue/pr deals with documentation (errors)
Projects
Project Board
Status: Backlog
Milestone
3.4.0
4 participants
@ueno @nhorman @t8m @mattcaswell