Shawn Walker wrote:
> https://cr.opensolaris.org/action/browse/pkg/swalker/pkg-bttf/webrev/
elf.py:
- Why /usr/ccs/lib and /usr/sfw/lib? These aren't in the runtime
linker's default runpath.
pkg/Makefile:
- I'd just set BUILDNUM=151. Otherwise, it looks like you have to build
on 151 in order to get the right version (which you'd be doing anyway,
but I think it's cleaner to fix it).
ips-incorporation.p5m:
- Why is cherrypy different than the others?
transforms/incorp:
- Shouldn't the copyright change be to "2010, 2011"?
setup.py:
- I'm not sure I understand why you didn't just revert all the changes
that involved pydates, rather than castrating them and patching them up
again. Does the result actually leave 5965 fixed? Seems like that
would actually be a very nice thing to have in the update branch. I
can try to help figure out what the right hg 1.3 incantations are, if
that'll help.
> The brand scripts are from changeset 2219 of the gate just before the
> dataset changes Mike made.
Be careful -- not everyone's 2219 is the same, and mine very definitely
isn't what yours is. Thankfully, your diff included the changeset id,
which let me figure out what was going on.
Thanks,
Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
