Skip to content

DEP: Update Neurodebian AFNI pin #557

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

Merged
merged 1 commit into from
Jun 13, 2017
Merged

Conversation

effigies
Copy link
Member

Builds seem to be breaking on this.

@yarikoptic Is this a bug, or does Neurodebian not keep old versions of packages around?

@yarikoptic
Copy link
Contributor

yes -- ATM there is no multiple versions support (we use reprepro). BUT if you really need specific previous version -- I could make it happen ... let me know

@chrisgorgo
Copy link
Contributor

Tested locally

@chrisgorgo chrisgorgo merged commit dbb18d4 into nipreps:master Jun 13, 2017
@effigies effigies deleted the update_afni branch June 13, 2017 18:51
@effigies
Copy link
Member Author

@yarikoptic Just making sure things are working as expected on your end. It's annoying that the package disappeared, but we weren't depending on the behavior of that specific version.

Perhaps we should consider setting a pin that's resilient to updates like that, but that's an issue for another day.

@satra
Copy link

satra commented Jun 14, 2017

@yarikoptic - afni really needs to be updated, especially post this year's fixes - i'm just making more noise :)

@yarikoptic
Copy link
Contributor

yeap -- I would also love to see it updated, so contributing to you @satra with the whining of my own (while listening to the talk on Pain)! Let's bring the level of complaints even higher -- @mih and @aqw . Time will come... time will come... I guess

@yarikoptic
Copy link
Contributor

although that issue is exactly the opposite to the one in question here @satra -- since desire is to have old pre-specified one :-P

@satra
Copy link

satra commented Jun 14, 2017

@yarikoptic - didn't we do that for fsl in simple workflow script - get any latest version available on a particular date?

@mih
Copy link

mih commented Jun 14, 2017 via email

@aqw
Copy link

aqw commented Jun 14, 2017

@yarikoptic We're doing a cluster upgrade here in early->mid july, so I can't touch AFNI until then.

AFNI updating stalled because there was an effort (lead by @yarikoptic IIRC) to track upstream more closely, do everything on github, and get the cmake build system upstreamed. I don't know where things stand right now with coordinating with upstream. But I'm 100% for it, and once cluster-fest is finished, I will happily dedicate time to it if it's more than a one man effort. Because there's a lot of complexity right now with brining that package up to date.

---Alex

@yarikoptic
Copy link
Contributor

@satra -- yeap, we did, and that is why I said that it could be done but with elevating some temporary curtain of security through obscurity ;)

@aqw there was effort indeed. upstream is on github, some tests are ran... but I haven't managed yet to replicate full upstream build structure within cmake, thus stalled... should not stop anyone update current debian-specific portion, if anything should be easier since upstream repo is now on github and some tests are already merged...

PS damn - they have no popcorn here

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.

6 participants