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
dnnl: run tests, fix install tree #79790
Conversation
With the new update to 1.2 in NixOS#79426, some the install tree coming out of cmake seems to have produced a duplicate copy of the nix tree.
# addition to the correct install; clean it up. | ||
postInstall = '' | ||
rm -r $out/nix | ||
''; |
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.
On master, we seem to have gained a superfluous install tree:
$ nix build -f . dnnl.{out,dev,doc}; tree result*
result
├── lib
│ ├── libdnnl.so -> libdnnl.so.1
│ ├── libdnnl.so.1 -> libdnnl.so.1.2
│ └── libdnnl.so.1.2
└── nix
└── store
└── wmf3lp0bxz8r0vnyfx2zrg9ynhhrn9al-dnnl-1.2
└── lib
├── cmake
│ └── mkldnn
├── libmkldnn.so -> libdnnl.so
├── libmkldnn.so.1 -> libdnnl.so.1
└── libmkldnn.so.1.2 -> libdnnl.so.1.2
result-1-dev
├── include
│ ├── dnnl_config.h
│ ├── dnnl_debug.h
│ ├── dnnl.h
│ ├── dnnl.hpp
│ ├── dnnl_types.h
│ ├── dnnl_version.h
│ ├── mkldnn_config.h
│ ├── mkldnn_debug.h
│ ├── mkldnn_dnnl_mangling.h
│ ├── mkldnn.h
│ ├── mkldnn.hpp
│ ├── mkldnn_types.h
│ └── mkldnn_version.h
├── lib
│ └── cmake
│ └── dnnl
│ ├── dnnl-config.cmake
│ ├── dnnl-config-version.cmake
│ ├── dnnl-targets.cmake
│ └── dnnl-targets-release.cmake
└── nix-support
└── propagated-build-inputs
result-2-doc
└── share
└── doc
└── dnnl
├── LICENSE
└── README
15 directories, 26 files
On this PR:
$ nix build -f . dnnl.{out,dev,doc}; tree result*
result
└── lib
├── libdnnl.so -> libdnnl.so.1
├── libdnnl.so.1 -> libdnnl.so.1.2
└── libdnnl.so.1.2
result-1-dev
├── include
│ ├── dnnl_config.h
│ ├── dnnl_debug.h
│ ├── dnnl.h
│ ├── dnnl.hpp
│ ├── dnnl_types.h
│ ├── dnnl_version.h
│ ├── mkldnn_config.h
│ ├── mkldnn_debug.h
│ ├── mkldnn_dnnl_mangling.h
│ ├── mkldnn.h
│ ├── mkldnn.hpp
│ ├── mkldnn_types.h
│ └── mkldnn_version.h
├── lib
│ └── cmake
│ └── dnnl
│ ├── dnnl-config.cmake
│ ├── dnnl-config-version.cmake
│ ├── dnnl-targets.cmake
│ └── dnnl-targets-release.cmake
└── nix-support
└── propagated-build-inputs
result-2-doc
└── share
└── doc
└── dnnl
├── LICENSE
└── README
9 directories, 23 files
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.
Likely an upstream bug. Perhaps some files are installed assuming a relative path.
A similar issue we had with also a cmake build in #76855.
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.
diff LGTM
[3 built, 1 copied (20.4 MiB), 3.6 MiB DL]
https://github.com/NixOS/nixpkgs/pull/79790
1 package built:
dnnl
With the new update to 1.2 in #79426, some
the install tree coming out of cmake seems to have produced a duplicate copy of
the nix tree.
Motivation for this change
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)