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

(#8062) Consider package epoch version when comparing yum package versions #224

Merged
merged 1 commit into from
Dec 13, 2011
Merged

(#8062) Consider package epoch version when comparing yum package versions #224

merged 1 commit into from
Dec 13, 2011

Conversation

jhelwig
Copy link
Contributor

@jhelwig jhelwig commented Nov 16, 2011

By including the epoch version in the version returned as the "latest"
available, we can now properly consider package updates where only the
epoch version has changed.

Signed-off-by: Jude Nagurney jude@pwan.org
Signed-off-by: Jacob Helwig jacob@puppetlabs.com

@pcarlisle
Copy link
Contributor

There's a comment in the install method "XXX: We don't actually deal with epochs here."

Is this wrong now (should be removed) or is it still the case for install and does that affect the functionality of this patch?

@jhelwig
Copy link
Contributor Author

jhelwig commented Dec 13, 2011

I'll take it out. I don't think we actually need to do anything special since yum can understand packages with epochs in either name-epoch:ver-rel.arch or epoch:name-ver-rel.arch form, and the former should just work if people wish to specify the epoch as part of the ensure value instead of as part of the name.

package { 'foo':
  ensure => '1:1.2.3',
  provider => 'yum',
}

vs

package { '1:foo':
  ensure => '1.2.3',
  provider => 'yum',
}

…sions

By including the epoch version in the version returned as the "latest"
available, we can now properly consider package updates where only the
epoch version has changed.

Signed-off-by: Jude Nagurney <jude@pwan.org>
Signed-off-by: Jacob Helwig <jacob@puppetlabs.com>
@pcarlisle pcarlisle merged commit 2be44d4 into puppetlabs:2.7.x Dec 13, 2011
melissa pushed a commit to melissa/puppet that referenced this pull request Mar 30, 2018
…ve_foreground_from_smf_manifest

(PA-46) Don't '--foreground' on startup on Solaris
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

3 participants