>>> Mark J. Nelson wrote:
>>> usr/src/lib/libinstzones/Makefile:
>>> usr/src/lib/libpkg/Makefile:

>>> _msg: Since you're already enumerating the object files in 
>>> Makefile.com, and you're preprocessing for message extraction, it 
>>> makes much more sense to handle _msg in Makefile.com, instead of 
>>> here.  And if you're really being clever, for libinstzones, you only 
>>> need to extract from zones_strings.h, using the -a flag for xgettext, 
>>> and not calling the preprocessor.

>> I ended up leaving this alone. I would have liked to have cleaned this
>> up, but I have already had way too much fun with these Makefiles and
>> have limited resources.  Thank you for the advice and I will file a bug
>> on this next week.

> I filed this bug and included your suggestion in the description:
> 6842322 cleanup _msg targets in Makefiles for libpkg and libinstzones

I'll want to see a webrev, to sign off on where you ended up for this wad.

>>> usr/src/cmd/svr4pkg/Makefile
>>> 30 (old file): I'm fine with you moving your message into the 
>>> SUNW_OST_OSCMD domain.  Where was SUNW_PKG_CMDS.po (and friends?)
>>> previously delivered?  Was there a SUNW0install or SUNW0adm package 
>>> that you'll need to either modify or obsolete?
>> When this fix integrates into the ON gate, the following files need to
>> be removed from the package SUNW0adm
>>     o usr/lib/locale/C/LC_MESSAGES/SUNW_PKG_CMDS.po
>>     o usr/lib/locale/C/LC_MESSAGES/SUNW_PKG_LIBPKG.po
>> Also, since SUNWpkgcmdr and SUNWpkgcmdu will now be delivered from ON,
>> so those packages need to be removed from the list of packages
>> integrated from the Legacy Install gate.  I believe that I will need
>> to coordinate with the ON gate staff as well as RE. 
> 
> To track this issue, I filed:
> 6842310 need to obsolete SVR4 packaging .po files and stop delivering
> SUNWpkgcmds*

Please bring this to the attention of your CRT advocate, so they can 
help you coordinate with whoever delivers the admin-nv pkgs, and with 
RE.  This fix should happen in the same build as your ON integration.

--Mark

Reply via email to