Debian package building #105
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Also related to #94 |
ervanalb
changed the title
zbanks
reviewed
Jul 31, 2019
| if [ -z "$4" ]; then | ||
| echo "You must specify a packaging version (e.g. 1) as the fourth argument" >&2 | ||
| exit 1 | ||
| fi |
There was a problem hiding this comment.
4 or 5 arguments is OK
zbanks
reviewed
Jul 31, 2019
| if [[ $TRAVIS_OS_NAME == 'osx' ]] | ||
| then | ||
| make bundle | ||
| elif [[ $TRAVIS_OS_NAME == 'linux' ]] |
There was a problem hiding this comment.
Can this be pushed into the yml? on.condition: "$TRAVIS_OS_NAME == linux" or does that not work
There was a problem hiding this comment.
This was the pattern already in most of the travis scripts, I don't really feel like changing it. TBH I don't understand travis's yml.
Fix paths in travis Pass in a version to build add -e flag to all scripts fix syntax, add doxygen fix path passed to debian.sh Add devscripts to travis for debian building Add fakeroot and attempt to deploy the artifact Default to 0~dev0 if no tag is present, and deploy the correct artifact (maybe) Fix some paths, maybe Flip conditional Add graphviz to dependencies for doxygen
ervanalb
changed the title
zbanks
approved these changes
Aug 2, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request migrates Travis to Bionic and gets it building a .deb file.
Once Debian packages are built by Travis, we can put them on a PPA through launchpad.
Related to #47 and #94