> Whoof.

Thanks for the review!

> Why is SUNWDTraceToolkit x86 only?  What about the SUNWgetting-started
> packages?  There are others, like SUNWbrasero and SUNWsexy-python, which at
> least have arch-specific bits in them, but shouldn't be available on both
> platforms anyway.  Do we know why they aren't?

In most if not all of the cases, it's because the consolidations in
question haven't delivered them.  Some of them like SUNWDTraceToolkit
just need to get built/repackaged (that one in particular could have
ARCH=all) but others may have some longer-term dependencies that need
to be addressed (such as the Compiz-related ones).

My expectation is that we'll be addressing this over time but some of
them should be fixed shortly.

> Need slim_install be available for sparc at all?  Or is it used for AI?
> Similar question for babel_install.

The AI team requested we supply them for now although I expect there
will be other clusters created in the future.

> We should probably pull slim_install and babel_install out of slim_cluster
> and redist_cluster so that we don't have two identical copies of each.
> Doesn't have to be this wad, though.

Okay.  I've been wanting to get rid of this duplication for awhile now
but I know the slim_cluster is still used by the Live CD folks for
testing purposes (haven't tested import times with recent bits to see
how much longer a full reditributable import takes).  I'll open up a
bug about this.

> Doesn't look like you got rid of i386/SUNWmdb?  Or is that just an artifact
> of the way the webrev was generated?  (I see the same for SUNWpicl.)

In order to import earlier builds, I wasn't going to get rid of the
i386 versions but in these two particular cases, the 105/common
versions were renamed from the 00/sparc and then modified - here are
the files in question

        A src/util/distro-import/105/common/SUNWmdb
        A src/util/distro-import/105/common/SUNWpicl
        A src/util/distro-import/105/common/SUNWsprot
        R src/util/distro-import/00/sparc/SUNWmdb
        R src/util/distro-import/00/sparc/SUNWpicl
        R src/util/distro-import/00/sparc/SUNWsprot

> For, say, SUNWnxge, can you put only the last line of the driver action
> under sparc_ONLY?  Otherwise, the driver action is exactly the same.

Yes, I'm made that change.

> In SUNWckr, mpt has a driver action defined.  For sparc, mpt is in SUNWpd,
> but that doesn't have a driver action defined.  In both cases, the aliases
> are in the base driver_aliases file, and not added via postinstall scripts.

Right - the i386 version was added earlier for upgrade purposes but for
SPARC, I've only added the explicit actions for those specified in the
postinstall scripts.  We still have to move all of the actions carried
out by the base files but I think that can wait for a different
commit.

> What's the change to no longer drop libXol from SUNWxwrtl?

As Alan mentioned, this was some cleanup that had not been seen since
we had not been importing the SPARC packages on a regular basis.
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to