npm install and build of forked github repo

拟墨画扇 提交于 2019-11-26 07:58:57

问题


This is not a totally new question, but I\'ve been looking around for a good while now and I\'m having trouble finding a solution.

I\'m using a module for my angular app called angular-translate. However, I\'ve had to make a few small modifications to the source code to get everything working the way I\'d like, and now I want to persist those changes on npm install. A colleague suggested that I fork the repo of the source code and point to my forked repo as a dependency, which I\'ve tried in these ways, e.g.

npm install https://github.com/myRepo/angular-translate
npm install https://github.com/myRepo/angular-translate/archive/master.tar.gz

The first gives me a directory like this with no build. Just a package.json, .npmignore, and some markdown files

-angular-translate
   .npmignore 
   .nvmrc
    CHANGELOG.md 
    package.json
    etc

The second npm install gives me the full repo, but again I don\'t get a build like when I use the command npm install angular-translate. I\'ve seen some dicussion of running the prepublish script, but I\'m not sure how to do this when installing all the modules. I\'ve also tried publishing the fork as my own module to the npm registry, but again I get no build, and I\'m not sure that\'s the right thing to do...

I apologise for my ignorance on the topic. I don\'t have a huge amount of experience with npm. Would love to get some feedback on this issue. It seems like it could be a common enough issue when modifications need to be made to a package\'s source code? Maybe there\'s a better solution? Thanks in advance for your help.


回答1:


Try npm install <ghusername>/<repoName>, where <ghUsername> is your GitHub username (without the @) and <repoName> is the name of the repository. That should correctly install it. You will most likely want to use the --save or --save-dev flag with the install command to save dependency in your package.json.

If that isn't working correctly, check the contents of your .npmignore file.

Don't panic if the install command takes a long time; installing from a git repository is slower than installing from the npm registry.


Edit:

Your problem is that in your case, dist/ is not committed to the repo (since it is in the .gitignore). That is where the actual code lives. dist/ is built from the files in src/ before the package is published to the npm registry, but dist/ is never committed to the repo.

It's ugly, but in this case you will have to remove dist/ from the .gitignore and then run:

npm run build
git add .
git commit
git push

(Ensure that you have run npm install first)

You then should be able to install from github.

There might be another way to do this using a postinstall script, but I'm not sure if that's possible; I've never tried it.




回答2:


Update for those using npm 5:

As of npm@5, prepublish scripts are deprecated.

Use prepare for build steps and prepublishOnly for upload-only.

I found adding a "prepare": "npm run build" to scripts fixed all my problems.




回答3:


To piggyback off of @RyanZim's excellent answer, postinstall is definitely a valid option for this.

Either do one of the following:

  1. Update the package.json in your forked repo to add a postinstall element to scripts. In here, run whatever you need to get the compiled output (Preferred).
  2. Update your package.json, and add a postinstall that updates the necessary directory in node_modules.

If you've forked another persons repository, then it might be worth raising an issue to illustrate the issue that installing their package through GitHub does not work as it does not provide the necessary means to build the script. From there, they can either accept a PR to resolve this with a postinstall, or they can reject it and you can do #2.




回答4:


prepare is the correct way

If you have a repository with source files but a "build" step is necessary to use it, adding a prepare script to that "package" does exactly what you want in all cases (as of npm 4).

prepare: Run both BEFORE the package is packed and published, on local npm install without any arguments, and when installing git dependencies.

You can even put your build dependencies into devDependencies and they will be installed before prepare is executed.

Here is an example of a package of mine that uses this method.


Problems with .gitignore

There is one issue with this option that gets many people. When preparing a dependency, Npm and Yarn will only keep the files that are listed in the files section of package.json.

One might see that files defaults to all files being included and think they're done. What is easily missed is that .npmignore mostly overrides the files directive and, if .npmignore does not exist, .gitignore is used instead.

So, if you have your built files listed in .gitignore like a sane person, don't list your built files in files, and don't use a .npmignore file, prepare will seem broken.

If you fix files to only include the built files or add an empty .npmignore, you're all set.


Original answer: https://stackoverflow.com/a/57503862/4612476




回答5:


Just use the command npm install git+https://git@github.com/myRepo/angular-translate.git. Thanks.



来源:https://stackoverflow.com/questions/40528053/npm-install-and-build-of-forked-github-repo

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!