Skip to content

Fix deprecated API use "SDist hooks violate the invariants of new-sdist" #249

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

Open
blackgnezdo opened this issue Sep 8, 2019 · 1 comment

Comments

@blackgnezdo
Copy link
Contributor

tensorflow-core-ops > configure (lib)
tensorflow-core-ops > [1 of 2] Compiling Main ( /tfhs/tensorflow-core-ops/Setup.hs, /tfhs/tensorflow-core-ops/.stack-work/dist/x86_64-linux/Cabal-2.4.0.1/setup/Main.o )
tensorflow-core-ops >
tensorflow-core-ops > /tfhs/tensorflow-core-ops/Setup.hs:53:7: warning: [-Wdeprecations]
tensorflow-core-ops > In the use of ‘sDistHook’
tensorflow-core-ops > (imported from Distribution.Simple, but defined in Distribution.Simple.UserHooks):
tensorflow-core-ops > Deprecated: "SDist hooks violate the invariants of new-sdist. Use 'autogen-modules' and 'build-tool-depends' instead."
tensorflow-core-ops > |
tensorflow-core-ops > 53 | , sDistHook = \p maybe_l h f -> case maybe_l of
tensorflow-core-ops > | ^^^^^^^^^
tensorflow-core-ops >
tensorflow-core-ops > /tfhs/tensorflow-core-ops/Setup.hs:57:21: warning: [-Wdeprecations]
tensorflow-core-ops > In the use of ‘sDistHook’
tensorflow-core-ops > (imported from Distribution.Simple, but defined in Distribution.Simple.UserHooks):
tensorflow-core-ops > Deprecated: "SDist hooks violate the invariants of new-sdist. Use 'autogen-modules' and 'build-tool-depends' instead."
tensorflow-core-ops > |
tensorflow-core-ops > 57 | sDistHook hooks (fudgePackageDesc l p) maybe_l h f
tensorflow-core-ops > | ^^^^^^^^^
tensorflow-core-ops > [2 of 2] Compiling StackSetupShim ( /root/.stack/setup-exe-src/setup-shim-mPHDZzAJ.hs, /tfhs/tensorflow-core-ops/.stack-work/dist/x86_64-linux/Cabal-2.4.0.1/setup/StackSetupShim.o )
tensorflow-core-ops > Linking /tfhs/tensorflow-core-ops/.stack-work/dist/x86_64-linux/Cabal-2.4.0.1/setup/setup ...

@blackgnezdo
Copy link
Contributor Author

@judah I can do something similar to your judah/proto-lens@3ad764e except I think we can skip cabal-1 compatibility as it shouldn't matter as much here.

blackgnezdo pushed a commit to blackgnezdo/tensorflow-haskell that referenced this issue Sep 8, 2019
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

No branches or pull requests

1 participant