* src/util/distro-import/172/redist_cluster (line 10)

 You removed 'include common/library:openldap' in with a load
 of java package removals. Just double checking that that was
 intended.

Yes, when a package changes from being process by the importer to being
delivered via native IPS, the import include of that package is removed
since it's no longer needed.  In this case, "openldap" moved from SFW
to Userland which took ownership of "openldap" and "SUNWopenldap".

* According to the attachment to bug #18849, both of these map to
 the same package.

 driver/network/hermon               driver/infiniband/connectx
 driver/network/mcxe                 driver/infiniband/connectx

 I know that's a valid possibility; I just wanted to make sure it
 was correct.

Yes, the contents of these packages were actually combined as part of

        7074517 hermon, mcxnex, and mcxe should be in one package

* src/pkg/manifests/group%2fsystem%2fsolaris-auto-install.p5m

 I see you are adding 'driver/network/ethernet/sxge' but
 'driver/network/sxge' wasn't there before. Just checking
 that that's correct.

Yes. For some reason, that package was missing from
"solaris-auto-install" which was an oversight.

 Similar question for 'system/io/infiniband/udapl' (there now) and
 'system/network/udapl' (wasn't there before).

 And again for 'system/zones/brand/brand-solaris' (there now) and
 'system/zones/brand/solaris' (wasn't there before).

These are actually dependencies now of other components and so they're
being pulled in.

* src/pkg/manifests/group%2fsystem%2fsolaris-large-server.p5m

 You are removing '-depend type=group fmri=driver/audio'
 but I don't see you adding 'system/io/audio'. Is that correct?

Yes. The package shouldn't be necessary on the "solaris-large-server"
and while it was earlier bring pulled in, it's no longer necessary.

Thanks for the careful review!
_______________________________________________
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to