Skip to content
This repository has been archived by the owner on Feb 1, 2021. It is now read-only.

Commit

Permalink
merged branch claylo/master (PR #58)
Browse files Browse the repository at this point in the history
This PR was merged into the master branch.

Commits
-------

d5e883f Do not display conflicting packages as dependencies in index.html

Discussion
----------

Don't display conflicts as deps

Pirum will now honor `conflicts` designation in a package dependency when building the index page, and not display a conflicting package _as_ a dependency.

See [package.xml 2.0 docs](https://pear.php.net/manual/en/guide.developers.package2.dependencies.php) for details on dependency conflicts.
  • Loading branch information
fabpot committed Oct 17, 2012
2 parents 3998d1b + d5e883f commit de69bfd
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion pirum
Original file line number Diff line number Diff line change
Expand Up @@ -485,9 +485,14 @@ pear install <?php echo $this->server->alias ?>/package_name-beta</code></pre>
$deps = array();
if (isset($package['deps']['required']['package'])) {
$deps = $package['deps']['required']['package'];
if (isset($deps['name'])) {
if (isset($deps['name']) && !isset($deps['conflicts'])) {
$deps = array($deps);
}
foreach ($deps as $i => $dep) {
if (isset($dep['conflicts'])) {
unset($deps[$i]);
}
}
}

$grps = array();
Expand Down

0 comments on commit de69bfd

Please sign in to comment.