On 11/28/11 16:13, Brock Pytlik wrote:
On 10/03/11 13:54, Brock Pytlik wrote:
Webrev:
https://cr.opensolaris.org/action/browse/pkg/bpytlik/18975-v1

Bug:
18975 resolve should be able to restrict which packages are used as
fall back

I've also changed our gate to use the new feature which gives a nice
speed boost when doing make packages.

Thanks for taking a look,
Brock
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
There was discussion on this bug, but no review of the code. Any chance
someone can take a look?

src/man/pkgdepend.1:
  line 179: s/be use/be used/

  line 181: The wording here seems a bit awkward at the end, could you
    try simplifying it a bit?

src/modules/publish/dependencies.py:
  line 1319: missing system_patterns and use_system from docstring;
    also curious as to whether system_patterns could imply use_system.

src/pkgdep.py:
  general: should we s/fmri/FMRI/ in error strings?  It is an acronym.

lines 603-611: These don't match current gate tip, I'll assume this just needs another resync.

src/tests/cli/t_pkgdep_resolve.py:
  lines 718, 724: s/self. output/self.output/

src/tests/pkg5unittest.py:
The downside of this change is that it makes it harder to find broken tests. I had originally purposefully not set these attributes as it made it failure obvious when a test checked for output or errout before running a command. Just an FYI; not an objection.

-Shawn
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to