search

TylerBrock / mongo-hacker

MongoDB Shell Enhancements for Hackers
tylerbrock.github.io/mongo-hacker
MIT License
1.79k stars 235 forks source link

Added an extra installation section on README.md file to install it from source #181

Closed quirinux closed 6 years ago

quirinux commented 6 years ago

Added an extra installation section on README.md file to install it from source clonning from github, it will help the people whith permission issues during the install process to avoid it.

It's not a final solution, but it's a way to get mongo-hacker install anyway on *nix OSes

quirinux commented 6 years ago

btw, this was the only way that I could get it installed on centos7 box

quirinux commented 6 years ago

Another suggestion here is to give up the npm install, firstly because it's not a nodejs application at all Secondly because a shell script can do the trick on *nix OSes, and a bat file for windows one Third, no symlink anywhere, just copy the file to .mongorc or even just load the mongo-hacker.js from mongorc file like described here: https://stackoverflow.com/questions/12062461/how-to-require-scripts-in-mongo-shell-cli

Need a hand with that? Do you believe it's something useful?

TylerBrock commented 6 years ago

Cool, thanks for the contribution. The original idea was that you'd clone the repository, run make install, then occasionally update it by doing a git pull followed by a subsequent make install.

Today that doesn't seem necessary as its being distributed as an NPM module but I'd like to rewrite this so that the hacks are concatenated by NodeJS (in a cross platform way) rather than using a batch file on Windows and a bash script on MacOS. The reason being that it is easier to maintain the code in a single place... also we don't really officially support windows.

Do you agree that this would be a decent approach given we have limited time to maintain things?

TylerBrock commented 6 years ago

Also, this is "must-install-from-source" problem is frequently encountered because you have installed node as root. I've got no problem on MacOS and Linux installing it when node is properly installed so that the user can write (install) npm packages so I'm not too keen on doing a lot of work to support that use case.

quirinux commented 6 years ago

You are right, npm handles better on that variety of OSes and is easier to use, I didn't make myself clear, sorry, well first things first, at that moment my setup was:

A CentOS VM hosted by a Win10 OS where some of my folders were mapped through virtual box sharing folder, aka vbsf protocol, which doesn't allow symbolic links at all. I think the another limitation is hoe the npm is installed by yum on CentOS, just root can install a package globally on the default npm installation, totally agree that some work around here could get it working properly.

anyways, thanks a lot for this great tool, it's helped me on my day to day activities and again, if you need a hand to maintain it just let me know

cheers