Stephen Lau writes:
> Rainer Orth wrote:
> > Btw., I'm currently working on
> >
> > 6414822 don't try to build packages with closed components
> >
> > again. The initial patch I sent to Mike consisted of just omitting any
> > packages with closed components in an open build. Right now, I'm almost
> > done splitting off closed components in several packages into their own
> > packages, as described at the start of this thread. Btw., do I need to
> > file a new bug for this work (splitting off packages), or can this be
> > handled with the same CR?
>
> I think that can happen under the same CR... Mike may have a better
> opinion though.
Fine. Compared to the original patch (just a few modifications to
pkgdefs/Makefile), the current one has grown quite a bit, though ;-)
> > It would be great if you could fix those issues for the closed tar balls.
>
> Sorry, just trying to clarify: is the problem in the Makefile with the
> untarring of the closed-bins, or is the problem actually in the creation
> process of the closed-bins tarball? If the former, then definitely
> request a sponsor (I'll be happy to), and we can fix the bug. If the
> latter, then it's a bug in the way I'm creating the closed-bins tarball...
It's both, unfortunately: one first needs to fix the closed-bins tarball
creation so the permissions there match what's in the package definitions,
and then make sure that the corrected permissions are correctly transfered
to the proto area.
I'll send off a message to request-sponsor, indicating you'll take this up.
> > Even after those changes and with my patches to split off closed components
> > (which, btw, ignore those files that can be opened as mentioned upthread,
> > but are not yet in the open tree)
>
> I'm moving these as part of the cfgadm plugins for sysctrl/ac. Should
> be ready next week.
Great, thanks.
> > Files missing from packages:
> >
> > T File Name Reloc/Sym name perm owner group
> > inode lnk maj min package(s)
> > ------------------------------------------------------------------------------------------------------------
> > f kernel/crypto/sparcv9/dprov - 755 - -
> > 246202 2 - - proto
> > f kernel/drv/dprov.conf - 644 - -
> > 0 1 - - proto
> > f kernel/drv/sparcv9/dprov - 755 - -
> > 246202 2 - - proto
> >
> > SUNWcryptoint
> >
> > Distributed in source; the only parts missing from SUNWcryptoint are
> > etc/certs/SUNWosnetSolaris and etc/crypto/certs/SUNWosnet. Perhaps those
> > should go into their own packages? Any suggestions for a name?
>
> Hrm... yeah, my feeling is those should probably be split into a
> separate package since the other parts of it are still interesting to
> open source - but the above two certs can not be distributed.
> cc'ing Darren Moffat for a more expert opinion...
Ok, that was my impression, too.
> > SUNWwrsu.u
> >
> > It seems strange now that all of the wrs code can be distributed in source
> > form with the only exception of the wrsm and wrsmd kmdb modules. Perhaps
> > this is an oversight which can be fixed?
>
> 6393468 move driver wrsmd to usr/src
> I'll add this to my wad for next week...
Good, this again reduces the size of my patch for 6414822 :-)
> I don't have a concrete answer for the other ones... hopefully Kupfer
> will weigh in. :)
Probably: we've already worked through several issues with this change.
Thanks.
Rainer
-----------------------------------------------------------------------------
Rainer Orth, Faculty of Technology, Bielefeld University
_______________________________________________
opensolaris-code mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/opensolaris-code
