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
WIP: haskellPackages.tensorflow: 2020-05-22 -> 2020-11-14 #111399
Conversation
Previously, the "ar command" in the global config of GHC in nixpkgs is simply "ar" instead of a proper absolute path in the nix store. This will result in an "ar: command not found" error when using GHC and cabal in a pure nix shell. This commit adds the patch and applies to all pre-9.0 versions. See output of ghc --info for "ar command" value.
Fixes build, adds support for TF 2.3
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.
It appears that everything other than tensorflow-proto
is marked broken.
Could you mark these as unbroken as well?
Ah yeah. Also although I'll mark this as WIP until that's resolved. |
36d8f4b
to
0b62665
Compare
@expipiplus1 I turned this to a Draft since we're waiting on upstream here, but feel free to flip it back to "Ready for review" when it is ready to be reviewed again. |
The upstream pull request got merged - could you update this one? |
915ff7a
to
f0391f1
Compare
56fa6fc
to
f3cb253
Compare
@expipiplus1 @cdepillabout What's needed to move this one forward? Would it help if I submitted an up-to-date pull request? |
@mikesperber Yes, that would be great! |
Yeah, might be easier to do that. Thanks!
…On Mon, Apr 12, 2021, 5:26 PM Dennis Gosnell ***@***.***> wrote:
@mikesperber <https://github.com/mikesperber> Yes, that would be great!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#111399 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGRJXGZUGMCDER7N7EHL5DTIK4E5ANCNFSM4W3CAJFQ>
.
|
@cdepillabout @expipiplus1 Done: |
Fixes build, adds support for TF 2.3
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)