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

make it easier to install nwm #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dominictarr
Copy link

I've added a bash script to make nwm easier to install.

It generates nwm.desktop, and has a script to start nwm inside xephyr.

./nwm init                #create nwm.desktop, and ~/.nwm/nwm-start
                              #this is called automatically by npm install
sudo ./nwm install    #move nwm.desktop to /usr/share/xsessions/
                              #will log to ~/.nwm/*.log
./nwm xephyr [size]  #start nwm in xephyr

It ended up uglier than I had hoped... because that is bash.
I had some problems since node can't see sudo on my coniguration.
but I've worked around it, hopefully in a way that will work for everyone.

Also, modified the scripts in the package.json so it would install for me,
I had to cd build; make to get ./build/Release/nwm.node. I added this to scripts.postinstall

cheers,

@mixu
Copy link
Owner

mixu commented Aug 7, 2012

Hi, thanks for this, but I'd prefer the installation to be done via actual packages for the specific OS, rather than bash scripts. The problem with bash is that it is hard to take into account the various environments, and if it breaks then it's easier for people to say "the installer is broken" vs "oh, maybe I need to check the path to node in the .desktop file".

I'm looking to target Ubuntu via .deb and Arch via AUR.

@dominictarr
Copy link
Author

good point.

@mixu mixu closed this Dec 12, 2014
@mixu mixu reopened this Dec 12, 2014
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

Successfully merging this pull request may close these issues.

2 participants