Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

make version usefull for reinstalling #45

Merged
merged 1 commit into from
Jan 20, 2017
Merged

Conversation

loosi
Copy link

@loosi loosi commented Jan 20, 2017

Overview

Right now the version parameter does only work at the first install. After that the "creates" param of exec disables any update/downgrade. I added a unless with test and grep to install the correct version.

We could use the self-update of composer to update to a specific version, but i dont know how that works for downgrades and it could possibly make you packages unable to work.

@Ma27 Ma27 added the in review label Jan 20, 2017
@loosi
Copy link
Author

loosi commented Jan 20, 2017

i dont really find the check that is responsible for that KeyError: key{linenumber} not found
I updated the spec test to make it work..but now idea where that errors comes from

@Ma27 Ma27 self-assigned this Jan 20, 2017
@Ma27 Ma27 added the bug label Jan 20, 2017
@Ma27 Ma27 added this to the 1.2.5 milestone Jan 20, 2017
@Ma27
Copy link
Collaborator

Ma27 commented Jan 20, 2017

seems legit, will test it locally and investigate what broke the build

@Ma27
Copy link
Collaborator

Ma27 commented Jan 20, 2017

the issue was caused by a broken puppetlint configuration. I fixed it on the master branch (5dff357). If you rebase master onto your branch and push this again, the build should be green (all tests passing on my machine)

@Ma27 Ma27 merged commit 1221074 into willdurand:master Jan 20, 2017
@Ma27 Ma27 removed the in review label Jan 20, 2017
@Ma27
Copy link
Collaborator

Ma27 commented Jan 20, 2017

merged locally, thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants