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
singularity: 3.0.1 -> 3.2.1 #63409
singularity: 3.0.1 -> 3.2.1 #63409
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.
- reviewed the diff and commit messages
- made sure ofBorg evals
- run nix-review without any failures
- run and tested the binaries
@GrahamcOfBorg build singularity |
Should this be back-ported to 19.03, given the fact that the old version seems to have a lot of bugs? |
What's the policy on back-porting? While 3.0.1 is buggy it at least isn't insecure (i.e., CVE-2019-11328 doesn't apply) and does have core functionality.
|
@jbedo If the bugs are severe enough and it does not break anything else or changes behavior a back port could be justified. I am not a singularity user so I would leave that decision the people actively using it. |
OK, I'm not sure the bugs are severe enough to warrant backporting, maybe @bbarker can comment. |
I don't have any knowledge of what the specific bugs are, other than the
one I'm experiencing, which may or may not be fixed by upgrading (still
need to test it). As it isn't a showstopper for me currently, I'm happy to
wait for now. What I should do is test the new version in a different
environment to see if that fixes the issue I had.
…On Tue, Jun 25, 2019 at 3:13 AM Justin Bedő ***@***.***> wrote:
OK, I'm not sure the bugs are severe enough to warrant backporting, maybe
@bbarker <https://github.com/bbarker> can comment.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#63409?email_source=notifications&email_token=AAG7XDVILIHKRZM57BFCSHDP4HASZA5CNFSM4HY35W3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYPIWMI#issuecomment-505318193>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAG7XDX7F4L4QJQCDMSUH3DP4HASZANCNFSM4HY35W3A>
.
--
Brandon Barker
brandon.barker@gmail.com
|
Motivation for this change
Version update, build process has changed slightly.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)