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.

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

autossh: support more architecture & add license #10405

Merged
merged 4 commits into from
Feb 5, 2025

Conversation

qiaojunfeng
Copy link
Contributor

No description provided.

Comment on lines 45 to 49
cat > ${WORKSPACE}/srcdir/LICENSE << EOF
Copyright (c) Carson Harding, 2002-2008.
All rights reserved.
EOF
install_license ${WORKSPACE}/srcdir/LICENSE
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uhm, that's not quite enough of a license, that's just saying all rights are reserved. Also, years are wrong? I'm looking at https://github.com/Autossh/autossh/blob/90a8c2f0129f6fe19ec26c7d0fdbab4bb468f476/autossh.c#L7-L22

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a licence in the beginning of the autossh.c source file. Looks MIT-ish.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In archlinux it's classified as "custom": https://archlinux.org/packages/extra/x86_64/autossh/. I haven't analysed the differences.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes I am taking the header of arch package
https://gitlab.archlinux.org/archlinux/packaging/packages/autossh/-/blob/main/LICENSE?ref_type=heads

Unfortunately, the original package has no license file, difficult to tell what it is.

Maybe I take exactly the header of the autossh.c file?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK Now I am just dumping the header of autossh.c as LICENSE

qiaojunfeng and others added 2 commits February 2, 2025 22:35
Co-authored-by: Mosè Giordano <765740+giordano@users.noreply.github.com>
@giordano giordano merged commit 579992e into JuliaPackaging:master Feb 5, 2025
19 checks passed
@qiaojunfeng qiaojunfeng deleted the autossh branch February 5, 2025 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants