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

dcm2niix: 1.0.20190410 -> 1.0.20190902 #71532

Merged
merged 1 commit into from Oct 22, 2019

Conversation

r-ryantm
Copy link
Contributor

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/dcm2niix/versions.

meta.description for dcm2niix is: '"dcm2niix DICOM to NIfTI converter"'.

meta.homepage for dcm2niix is: '"https://www.nitrc.org/projects/dcm2nii"

Release on GitHub

Compare changes on GitHub

Checks done (click to expand)
Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
dcm2niix

Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/8y0j1zl3qabidls1aw0d45ay317bkyw3-dcm2niix-1.0.20190902 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)

Or, build yourself:

nix-build -A dcm2niix https://github.com/r-ryantm/nixpkgs/archive/d2ea06ed85f8e17c737a1a6454790ddea291a4fe.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/8y0j1zl3qabidls1aw0d45ay317bkyw3-dcm2niix-1.0.20190902
ls -la /nix/store/8y0j1zl3qabidls1aw0d45ay317bkyw3-dcm2niix-1.0.20190902/bin

cc @ashgillman for testing.

Semi-automatic update generated by
https://github.com/ryantm/nixpkgs-update tools. This update was made
based on information from
https://repology.org/metapackage/dcm2niix/versions
Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

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

nix-review passes on NixOS
diff LGTM
executable seem to work

[3 built, 2 copied (1.2 MiB), 0.2 MiB DL]
https://github.com/NixOS/nixpkgs/pull/71532
1 package were build:
dcm2niix

@ashgillman
Copy link
Contributor

I'm afraid I will not be able to text this for about a week. I'm on conference without my Nix PC. I'm happy with @jonringer 's testing for this to be accepted in the meantime.

Copy link
Contributor

@ashgillman ashgillman left a comment

Choose a reason for hiding this comment

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

Approved based on @jonringer 's tests

@jonringer
Copy link
Contributor

it's a patch version, i don't think it's too high risk

@jonringer jonringer merged commit a9551de into NixOS:master Oct 22, 2019
@r-ryantm r-ryantm deleted the auto-update/dcm2niix branch November 1, 2019 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants