Skip to content
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

jsdeps for egg_info ? #44

Open
zerline opened this issue Mar 4, 2019 · 1 comment
Open

jsdeps for egg_info ? #44

zerline opened this issue Mar 4, 2019 · 1 comment

Comments

@zerline
Copy link
Contributor

zerline commented Mar 4, 2019

Hi
Is jsdeps really necessary for building egg_info? On my project, that resulted in computing jsdeps twice, one for egg_info, the second for build_py, and I wonder why this is at all required for egg_info ?

cf in setup.py :

 'cmdclass': {
        'build_py': js_prerelease(build_py),
        'egg_info': js_prerelease(egg_info),
        'sdist': js_prerelease(sdist, strict=True),
        'jsdeps': NPM,
    },
@vidartf
Copy link
Member

vidartf commented Mar 6, 2019

Hi,
For reference, in widget-ts-cookiecutter we only set bdist_egg. Maybe would be a good replacement here?

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

2 participants