On Fri, Aug 01, 2008 at 06:57:37PM -0700, Bart Smaalders wrote:

> http://cr.opensolaris.org/~barts/2589/

.hgignore:

  - I wonder if perhaps we shouldn't have one pattern to eliminate all
    prototype files?

file.py:

  - line 90: Now that you removed the setting of ohash, you should change
    this reference to it to be orig.hash.

generic.py:

  - not sure I understand the purpose of this change (the comment change I
    get).

Missing change to TarBundle.py?

t_pkg_install_basics.py:

  - It would be useful to have a test where you touch /tmp/libc.so.1 to
    some other time than is in the timestamp and confirm that it fails
    verification.

We probably want to preserve timestamps on *all* .py files, or at least all
those which have .pyc files right next to them.  Getting SUNWipkg is great,
but everything else would be better.  You could do this in the Makefile if
you blindly did all .py files.  The .py-only-if-.pyc bit would need logic
in solaris.py.

publish.py:

  - line 274: use the else clause on the for loop to do something when you
    fall off the end, rather than breaking out:

        for pattern in files:
            if fnmatch(...):
                break
        else:
            del action.attrs["timestamp"]

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

Reply via email to