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
Revert "python: gym: 0.15.3 -> 0.15.4" #77049
Conversation
This reverts commit 1ae98ee.
to prevent accidental bumpage
@GrahamcOfBorg build python27Packages.gym python37Packages.gym python37Packages.roboschool python38Packages.gym python38Packages.roboschool |
Please add an assert instead, that will be more effective.
…On Mon, 6 Jan 2020, 13:35 Jon, ***@***.***> wrote:
@GrahamcOfBorg <https://github.com/GrahamcOfBorg> build
python27Packages.gym python37Packages.gym python37Packages.roboschool
python38Packages.gym python38Packages.roboschool
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#77049?email_source=notifications&email_token=AAQHZ37CBSDMTBSBEASKA2TQ4MQSDA5CNFSM4KDC7NZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIFKIVQ#issuecomment-571122774>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAQHZ37I7D6PEAWVJSEZZATQ4MQSDANCNFSM4KDC7NZQ>
.
|
This is intended to prevent the pypi update script from targeting this. Even with an assert, this file will be updated. The other option would be to do FetchFromGitHub |
Yes but it would fail on eval, that way it would be clear. When updating I often take all nix files, so this would not help. |
the file may be passed to the update script, but the update script skips files which has nixpkgs/pkgs/development/interpreters/python/update-python-libraries/update-python-libraries.py Line 242 in 15695df
|
bump @FRidh what do you think about using fetchFromGitHub? |
update in #80024 makes this unnecessary |
Motivation for this change
noticed it was broken, froze/renamed it to prevent accidental bumpage
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @
failures broken on master