-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add Debian packaging policy and sample package documents. #62
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @futurejones for the policy, it looks good to me.
cc: @tomerd for additional review |
What kind of warnings are we seeing here? |
E: swiftlang: dir-or-file-in-opt opt/bin/
N:
N: Debian packages should not install into /opt, because it is reserved for
N: add-on software.
N:
N: Refer to Filesystem Hierarchy Standard (/opt : Add-on application
N: software packages) for details.
N:
N: Severity: error
N:
N: Check: files/hierarchy-standard
W: swiftlang: file-in-unusual-dir libexec/swift/bin/clang
N:
N: This file or symbolic link is in a directory where files are not
N: normally installed by Debian packages.
N:
N: Severity: warning
N:
N: Check: files/hierarchy-standard |
@drexin pointed out we might be able to use |
@shahmishal, I was looking at the same thing. |
@futurejones Did |
@shahmishal I was looking at the
Currently they are being linked to |
I dont think so, do you see any benefits on linking directly? |
I don't know of any benefit, I was just wondering as to the reasoning of creating symlinks to symlinks. |
@futurejones This is ready to be merged once we update the doc to contain the info about using |
@shahmishal @tomerd @jblache |
@futurejones sorry to hear that the work on #90 had left this impression, it was not the intention and my guess is that what we have here is a communication breakdown of some sorts the work on #90 started since this PR and more generally debian packaging had no progress for many months. there is a strong desire to make progress on offering debian packages along side the RPMs swift.org is already offering. very much like this PR, #90 is a proposal - a PR that is open for community review, feedback and discussion. this project welcomes the involvement of everyone that cares to contribute to the effort including Apple employees and community members |
Added a Debian Policy document to outline the procedure to get packages accepted into the official Debian repositories. I have included links to reference docs for further information.
Included in the PR are sample build instructions for a ubuntu/focal deb package for initial testing. The test results have been reported in SR-15515.
While testing I created several packages to test the alternate installation locations as discussed in previous PR's
opt
andlibexec
.It appears that like
fedora
,debian
does not allow installation in theopt
directory.Installation into the
libexec
also caused many warnings.