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

Introduced a dependency mapping for != dependency #221

Merged
merged 2 commits into from May 23, 2012
Merged

Introduced a dependency mapping for != dependency #221

merged 2 commits into from May 23, 2012

Conversation

svenihoney
Copy link
Contributor

There exist gems in the wild that have a dependency !=
Example is sprocket gem (at least version 2.1.3). Use a mapping
that prevents dpkg to choke on != and appended it to the conflicts.

There exist gems in the wild that have a dependency !=
Example is sprocket gem (at least version 2.1.3). Use a mapping
that prevents dpkg to choke on != and appended it to the conflicts.
@@ -261,6 +261,13 @@ def fix_dependency(dep)
nextversion[l-1] = 0
nextversion = nextversion.join(".")
return ["#{name} (>= #{version})", "#{name} (<< #{nextversion})"]
elsif (m = dep.match(/(\S+)\s+\(!= (.+)\)/))
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this really what is meant by !=?

As an alternate, what if instead of adding a dependency, the '!=' dependency adds a 'Conflicts' entry?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dammit, uploaded the wrong patch. Stay tuned, will upload the correct one after dinner...

elsif (m = dep.match(/(\S+)\s+\(!= (.+)\)/))
# Append this to conflicts
self.conflicts += [dep.gsub(/!=/,"=")]
return []
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@jordansissel
Copy link
Owner

grr, wrong click ;)

jordansissel added a commit that referenced this pull request May 23, 2012
Introduced a dependency mapping for != dependency using Conflicts in deb instead of a Depends
@jordansissel jordansissel merged commit ac5a399 into jordansissel:master May 23, 2012
prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 18, 2014
Introduced a dependency mapping for != dependency using Conflicts in deb instead of a Depends
prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 27, 2014
Introduced a dependency mapping for != dependency using Conflicts in deb instead of a Depends
jordansissel added a commit that referenced this pull request Apr 24, 2015
Introduced a dependency mapping for != dependency using Conflicts in deb instead of a Depends
jordansissel added a commit that referenced this pull request Jun 20, 2016
Introduced a dependency mapping for != dependency using Conflicts in deb instead of a Depends
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

2 participants