suggestion related to package dependencies.

[heartsucker]

suggestion related to package dependencies.

[heartsucker]

@Ashishjshetty Your idea is similar to what I was considering implementing, but I think this would start creeping into a few too many features/flags. Right now node-deb uses reasonable (IMO) defaults for everything but provides the ability to override everything else via templates. I think I'm going to decline your offer to add this feature at the moment.

[heartsucker]

One option would be to use the --template-control option to pass in a template for the control file with the dependencies hard coded and to not use the {{ node_deb_package_dependencies }} notation. This will avoid injecting nodejs (or anything else from package.json) and still let you specify your dependencies.

I think there is a fix I can make that won't break backwards compatibility, but I won't have time to implement it for a few days at least.

[Ashishjshetty]

Sure, if there is anything I can do , let me know, I will look into what
you suggest ed. Thank you.

On Oct 14, 2016 5:06 PM, "heartsucker" [email protected] wrote:

One option would be to use the --template-control option to pass in a
template for the control file with the dependencies hard coded and to not
use the {{ node_deb_package_dependencies }} notation. This will avoid
injecting nodejs (or anything else from package.json) and still let you
specify your dependencies.

I think there is a fix I can make that won't break backwards
compatibility, but I won't have time to implement it for a few days at
least.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#26 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGISJBEoWgvlFOTyJGcWI2ZrPKwwp4Zlks5qz2k2gaJpZM4KW58P
.

[Ashishjshetty]

hi @heartsucker ,
I was thinking maybe we can add another option something like node_deb.no_node = true in package.json and check if is set ,ignore nodejs dependency and in all other cases inject nodejs dependency which will allow backwards compatibility. What do you think?, If you want i will make the necessary changes and send a PR.

[Ashishjshetty]

yes i thought the same, will go with template control option for now, Thank You.