search

Differential / meteor-vulcanize

Vulcanize html imports in your meteor app.
32 stars 15 forks source link

VULCANIZE=true not working on Meteor for Windows #6

Open AndreasGalster opened 9 years ago

AndreasGalster commented 9 years ago

I'm using the latest Windows preview build. Says the term VULCANIZE=true is not recognized as the name of a cmdlet, script or file, etc.

Maybe make 'meteor VULCANIZE=true' possible instead?

bangonkali commented 9 years ago

You have to check the batch file syntax for creating environment variable in Windows or go to start then type environment variables on the search. There's an environment variable manager on Windows wherein you can add new.environment variables using the GUI.

AndreasGalster commented 9 years ago

And how would I set te environment variable? I know I need to reference my variable to vulcanize but where is the path to vulcanize? In the project itself in the package?

schnie commented 9 years ago

I don't have a windows machine handy so I can't try anything at the moment but maybe try using the SET command somehow. Not sure if it can be used in a one liner, may require a little batch file.

And how would I set te environment variable? I know I need to reference my variable to vulcanize but where is the path to vulcanize? In the project itself in the package?

— Reply to this email directly or view it on GitHub https://github.com/Differential/meteor-vulcanize/issues/6#issuecomment-122749978 .

AndreasGalster commented 9 years ago

Setting the actual variable with command or environment variable is noth the problem I'm having. I just don't know what I actually need to put in the variable. I'm guessing variable is VULCANIZE but I don't know what to put in the value.

I've tried setting it to VULCANIZE / path to my npm folder/ vulcanize.cmd in npm folder.

schnie commented 9 years ago

The value just needs to be "true" On Jul 21, 2015 12:36 AM, "AndreasGalster" notifications@github.com wrote:

Setting the actual variable with command or environment variable is noth the problem I'm having. I just don't know what I actually need to put in the variable. I'm guessing variable is VULCANIZE but I don't know what to put in the value.

I've tried setting it to VULCANIZE / path to my npm folder/ vulcanize.cmd in npm folder.

— Reply to this email directly or view it on GitHub https://github.com/Differential/meteor-vulcanize/issues/6#issuecomment-123154581 .

bangonkali commented 9 years ago

Numeric 1 will do instead of "true". But it actually works with string true because it will be parsed as true by javascript anyway. On Jul 21, 2015 6:42 PM, "Greg Neiheisel" notifications@github.com wrote:

The value just needs to be "true" On Jul 21, 2015 12:36 AM, "AndreasGalster" notifications@github.com wrote:

Setting the actual variable with command or environment variable is noth the problem I'm having. I just don't know what I actually need to put in the variable. I'm guessing variable is VULCANIZE but I don't know what to put in the value.

I've tried setting it to VULCANIZE / path to my npm folder/ vulcanize.cmd in npm folder.

— Reply to this email directly or view it on GitHub < https://github.com/Differential/meteor-vulcanize/issues/6#issuecomment-123154581

.

— Reply to this email directly or view it on GitHub https://github.com/Differential/meteor-vulcanize/issues/6#issuecomment-123259630 .

AndreasGalster commented 9 years ago

Tried both Powershell and normal CMD. Git Bash doesn't work with meteor unfortunately. Any thoughts?

screenshot 2015-07-21 20 58 33

schnie commented 9 years ago

If you already have it set in your environment variables through the GUI, you can go ahead and omit it from the command to start meteor. Just do a regular old meteor --settings.json... hopefully.

On Tue, Jul 21, 2015 at 9:03 AM, AndreasGalster notifications@github.com wrote:

Tried both Powershell and normal CMD. Git Bash doesn't work with meteor unfortunately. Any thoughts?

[image: screenshot 2015-07-21 20 58 33] https://cloud.githubusercontent.com/assets/3836411/8801183/4748343a-2feb-11e5-8d62-ce826e9e9b59.png

— Reply to this email directly or view it on GitHub https://github.com/Differential/meteor-vulcanize/issues/6#issuecomment-123297878 .

AndreasGalster commented 9 years ago

Seems like nothing is happening. I've re-installed the package just to be sure I have the latest version, it didn't help unfortunately. It's not giving any errors either, so I'm guessing it's not doing anything at all.

ThaumRystra commented 9 years ago

Setting VULCANIZE to true in the system variables (not user variables) in the windows environmental variables GUI worked for me. All the imports get vulcanized correctly.