Skip to content

auto-build-rfc#38

Closed
hemanth wants to merge 1 commit into
yarnpkg:masterfrom
hemanth:auto-build-rfc
Closed

auto-build-rfc#38
hemanth wants to merge 1 commit into
yarnpkg:masterfrom
hemanth:auto-build-rfc

Conversation

@hemanth

@hemanth hemanth commented Dec 26, 2016

Copy link
Copy Markdown

//cc @samccone @kittens

#11

@ljharb

ljharb commented Dec 26, 2016

Copy link
Copy Markdown

@hemanth if you want a postinstall script, couldn't you add one under "postinstall"?

@hemanth

hemanth commented Dec 27, 2016

Copy link
Copy Markdown
Author

Yes, that's what I meant in yarn#1416.

@Daniel15

Copy link
Copy Markdown
Member

Do any other packages automatically build when you run npm install? That feels a bit too magical for me 😛

What is the purpose of this change?

@hemanth

hemanth commented Jan 17, 2017

Copy link
Copy Markdown
Author

For making things easier with git clone and npm link.

@Daniel15

Daniel15 commented Jan 17, 2017

Copy link
Copy Markdown
Member

Sure, but what's the end goal? 😃

For a user of Yarn that wants to use the latest version, we have nightly builds available to use (https://yarnpkg.com/en/docs/nightly)
For someone building on Yarn itself, you'd need to know how to build Yarn, so I'm not sure that automating just the first build will actually help, unless people end up understanding exactly why it happens. You still need to know the build command in order to test things.

I'm not against the change at all, just curious as to its motivation.

@hemanth

hemanth commented Jan 17, 2017

Copy link
Copy Markdown
Author

I'm not against the change at all, just curious as to its motivation.

Laziness being the main motivation ;)

On a serious not, say someone is making a PR to yarn repo, if npm link does the required, it would be one step lesser and easier?

@Daniel15

Copy link
Copy Markdown
Member

say someone is making a PR to yarn repo, if npm link does the required, it would be one step lesser and easier?

For what it's worth, I don't use npm link when building on Yarn. I just run it with the exact absolute path (for example, c:\src\yarn\bin\yarn or /usr/local/src/yarn/bin/yarn). It makes it immediately obvious exactly which version of Yarn I'm running :)

@doug-wade

doug-wade commented Feb 1, 2017

Copy link
Copy Markdown

@hemanth
Why aren't we proposing use of the npm prepare script for this? It seems to me that following the npm pattern would be better for interoperability, and make this easier.

@Daniel15

Do any other packages automatically build when you run npm install?

Yes, all of them. When you run npm link, it runs the prepare script (prepublish in older npms)

@bestander

Copy link
Copy Markdown
Member

done via yarnpkg/yarn#3553

@bestander bestander closed this Jun 29, 2017
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.

5 participants