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
bftools: init at 5.9.2 #53769
bftools: init at 5.9.2 #53769
Conversation
d642db9
to
a034c2c
Compare
Thanks for the great feedback, @veprbl! I've pushed corrections for all resolved conversations |
Also you don't need unzip: diff --git a/pkgs/applications/science/biology/bftools/default.nix b/pkgs/applications/science/biology/bftools/default.nix
index aa30cbfc5ca..c45a577fc60 100644
--- a/pkgs/applications/science/biology/bftools/default.nix
+++ b/pkgs/applications/science/biology/bftools/default.nix
@@ -1,12 +1,12 @@
-{ stdenv, lib, makeWrapper, fetchurl, unzip, openjdk }:
+{ stdenv, lib, makeWrapper, fetchzip, unzip, openjdk }:
stdenv.mkDerivation rec {
name = "bftools-${version}";
version = "5.9.2";
- src = fetchurl {
+ src = fetchzip {
url = "http://downloads.openmicroscopy.org/bio-formats/${version}/artifacts/bftools.zip";
- sha256 = "02x8p07z7js41kgiblkr9k8v9v4ka8rm7pg3f1vy9s5p6hmpfvnx";
+ sha256 = "08lmbg3kfxh17q6548il0i2h3f9a6ch8r0r067p14dajhzfpjyqj";
};
installPhase = ''
@@ -20,8 +20,6 @@ stdenv.mkDerivation rec {
done
'';
- buildInputs = [ unzip ];
-
nativeBuildInputs = [ makeWrapper ];
meta = with stdenv.lib; {
|
Co-Authored-By: tbenst <tbenst@gmail.com>
Co-Authored-By: tbenst <tbenst@gmail.com>
Merge branch 'bioformattools' of github.com:/tbenst/nixpkgs into bioformattools
without unzip, I get Made all other changes except @symphorien's idea, as I'm not sure how to change the directory structure without breaking the software. |
@tbenst This is why my snippet switches |
@vberger doh! fixed |
Switching the directory structure would require you to patch the shell scripts. Something like: substituteInPlace "$out"/bin/bfconvert \
--replace "\$BF_DIR" "$out"/libexec |
thanks! I got stuck for now and can't spend more time on this for a few days:
Edit: jk figured it out! |
@GrahamcOfBorg build bftools |
@GrahamcOfBorg build bftools |
Motivation for this change
This package is useful for microscopy. First pull request here so feedback most welcome!
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)I tested bfconvert, which is the only tool I use, and this worked properly for converting a .oir file to a .ome.btf. The other executables display "--help" properly but were not otherwise tested.