-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
a994f2f
commit 3c32be0
Showing
1 changed file
with
2 additions
and
2 deletions.
There are no files selected for viewing
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
3c32be0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't get it, on the website it says the latest LTS (and indeed the one we use) is 1.565.1. Why does the image use 1.576?
3c32be0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh that is a stupid mistake - fat fingering. It takes it from the stable package so that is totally incorrect. Sorry will fix (ie those numbers are misleading).
3c32be0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mdirkse this is using the debian packaging - and as the builds on docker as based on pull requests to their official build system - it isn't always able to keep up - but in this case I just typed the wrong number in :(
Trying to think of a better way of tackling this... thanks for pointing this out.
3c32be0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be better to do this:
after installing jenkins package instead of hardcoded the lts version number?
3c32be0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3c32be0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vdupain what about:
wget ... jenkins.deb
dpkg -i --force-depends $VERSION.deb
or similar:
dpkg -i downloaded_package.deb
apt-get -f install
these both are bad - first one uses force, second one will actually fail first (so need to ignore that) - but allow the version of the deb to be installed to be explicit ?
Could you open this as an issue on the repo (vs a comment here).