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 bliss package #42897
add bliss package #42897
Conversation
|
||
nativeBuildInputs = [ unzip doxygen ]; | ||
|
||
unpackCmd = '' |
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.
this shouldn't be needed - unzip has a hook to do this.
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.
done
url = "http://scip.zib.de/download/bugfixes/scip-5.0.1/bliss-0.73.patch"; | ||
sha256 = "815868d6586bcd49ff3c28e14ccb536d38b2661151088fe08187c13909c5dab0"; | ||
}; | ||
src = [ blissSrc blissPatch ]; |
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.
src shuold just be blisssrc
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.
done
esac | ||
''; | ||
|
||
patchPhase = '' |
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.
Just put blissPatch in patches
- this makes things more composable.
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.
done
|
||
nativeBuildInputs = [ unzip doxygen ]; | ||
|
||
patches = [ blissPatch ]; |
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.
I'd prefer to have the patch here directly (patches = [ (fetchurl { ...
). Because blissPatch
would be passed to the builder which doesn't use it. If you do want to assign variables, use a let in
at the beginning
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.
done
And you need to finally learn how to squash multiple commits into one, look it up online. |
@GrahamcOfBorg build bliss |
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.
squash into one commit now done
|
||
nativeBuildInputs = [ unzip doxygen ]; | ||
|
||
patches = [ blissPatch ]; |
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.
done
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: bliss Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: bliss Partial log (click to expand)
|
One more thing: Change the commit message to "bliss: init at 0.73" |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)