Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants