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

Update Emscripten SDK cleanly #473

Merged
merged 1 commit into from
May 21, 2019
Merged

Update Emscripten SDK cleanly #473

merged 1 commit into from
May 21, 2019

Conversation

ilammy
Copy link
Collaborator

@ilammy ilammy commented May 21, 2019

Our CI builds started failing recently.

For some reason building with Emscripten changes some files in the SDK which leads to errors when trying to update with git pull:

Updating 4eeff61..751355a
error: Your local changes to the following files would be overwritten by merge:
        binaryen-tags.txt
        emscripten-tags.txt
        upstream/lkgr.json

Please, commit your changes or stash them before you can merge.
Aborting

Add a git reset before doing the pull to discard any local changes and ensure a clean merge with the updated version.

For some reason building with Emscripten changes some files in the
SDK which leads to errors when trying to update with "git pull":

    Updating 4eeff61..751355a
    error: Your local changes to the following files would be overwritten by merge:
            binaryen-tags.txt
            emscripten-tags.txt
            upstream/lkgr.json

    Please, commit your changes or stash them before you can merge.
    Aborting

Add a "git reset" before doing the pull to discard any local changes
and ensure a clean merge with the updated version.
@ilammy ilammy added the infrastructure Automated building and packaging label May 21, 2019
@ilammy ilammy requested review from vixentael and Lagovas May 21, 2019 09:43
@ilammy ilammy merged commit 5ea9e31 into cossacklabs:master May 21, 2019
@ilammy
Copy link
Collaborator Author

ilammy commented May 21, 2019

Android builds started failing recently as well. We'll address that in a separate PR.

@ilammy ilammy deleted the emsdk branch May 21, 2019 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Automated building and packaging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants