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

pack.sh: support building outside of Git #208

Merged
merged 1 commit into from Jul 26, 2019

Conversation

mortend
Copy link
Member

@mortend mortend commented Jul 25, 2019

Don't terminate the script if a git command fails.

This fixes building .tar.gz files downloaded from GitHub, reported broken in
https://forums.fusetools.com/t/unable-to-compile-uno-fatal-not-a-git-repository/7817/1

Don't terminate the script if a git command fails.

This fixes building .tar.gz files downloaded from GitHub, reported broken in
https://forums.fusetools.com/t/unable-to-compile-uno-fatal-not-a-git-repository/7817/1
@mortend mortend changed the base branch from master to release-1.12 July 25, 2019 08:42
scripts/pack.sh Show resolved Hide resolved
@mortend mortend merged commit dc591b1 into fuse-open:release-1.12 Jul 26, 2019
@mortend mortend deleted the no-git-build branch July 26, 2019 13:35
mortend added a commit to mortend/uno that referenced this pull request Sep 30, 2019
* When building on AppVeyor:

    Drop prerelease-suffix on tagged commits (official releases),
    or use canary-suffix on when building the master-branch,
    or use rc-suffix on when building a release-branch.

* When building from Git repo:

    Get commit SHA from local git-repo.
    Always use dev-suffix.

* Otherwise:

    Always use dev-suffix.

Following up a little bit on fuse-open#208, I think this new logic is less awkward.

I also added a canary-suffix on master-builds, and the idea is to set up a
canary-channel on NPM where we can easily publish the latest master-build.
@mortend mortend mentioned this pull request Sep 30, 2019
mortend added a commit to mortend/uno that referenced this pull request Oct 7, 2019
* When building on AppVeyor:

    Drop prerelease-suffix on tagged commits (official releases),
    or set canary-suffix when building the master-branch,
    or set rc-suffix when building a release-branch.

* When building from Git repo:

    Get commit SHA from local git-repo.
    Always set dev-suffix.

* Otherwise:

    Always set dev-suffix.

Following up a little bit on fuse-open#208, I think this new logic is less awkward.

I also added a canary-suffix on master-builds, and the idea is to set up a
canary-channel on NPM where we can easily publish the latest master-build.
mortend added a commit to mortend/uno that referenced this pull request Nov 11, 2019
* When building on AppVeyor:

    Drop prerelease-suffix on tagged commits (official releases),
    or set canary-suffix when building the master-branch,
    or set rc-suffix when building a release-branch.

* When building from Git repo:

    Get commit SHA from local git-repo.
    Always set dev-suffix.

* Otherwise:

    Always set dev-suffix.

Following up a little bit on fuse-open#208, I think this new logic is less awkward.

I also added a canary-suffix on master-builds, and the idea is to set up a
canary-channel on NPM where we can easily publish the latest master-build.
mortend added a commit to mortend/uno that referenced this pull request Nov 14, 2019
This drops $BRANCH and $SUFFIX, and simplifies logic (fuse-open#208).

When updating an NPM package, the version field must be updated in
package.json, and all changes to package.json should be committed in Git.

This means that publishing an NPM package, pre-release or not, will always
involve updating package.json, committing, and pushing a tag in Git.

This also means that pack.sh should not manipulate version info read from
package.json, as it used to do in pre-NPM times for legacy reasons.
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.

None yet

2 participants