On 01/30/2012 04:32 PM, Alan Coopersmith wrote:
On 01/30/12 10:28 AM, Rich Burridge wrote:
Hi all,

Could I please get a code review for the changes for:

6886388 Ship automake-1.11.2
http://monaco.us.oracle.com/detail.jsf?cr=6886388

7127352 autoconf should be updated to the latest version
http://monaco.us.oracle.com/detail.jsf?cr=7127352

7132411 Various Userland components will need adjusting when autoconf
and automake are updated
http://monaco.us.oracle.com/detail.jsf?cr=7132411

Webrev is at:

http://jurassic.us.oracle.com/~richb/autotools-update-v1/

Besides the discussion we already had about changing the new package name
from -1112 to just -111, I also noticed:

- I assume the doc removal from -110 is because they're now being shipped
  in -111 instead.   It looks like you could also drop a couple more dir
  entries from -110 now that are also no longer needed:
     dir path=usr/share/doc
     dir path=usr/share/doc/automake

Indeed. So changed. A new webrev with this change
(and the change of the automake 1.11.2 package name from:

  pkg:/developer/build/automake-1112    to
  pkg:/developer/build/automake-111

which is mentioned above) can be found at:

  http://jurassic.us.oracle.com/~richb/autotools-update-v3/

I've rebuilt those two components in my x86 workspace. Final package
manifests can be found at:

/net/stard.us.oracle.com/tank/ws/UL/autotools-update/components/automake/automake-1.10/build/manifest-i386-automake.published
/net/stard.us.oracle.com/tank/ws/UL/autotools-update/components/automake/automake-1.11.2/build/manifest-i386-automake.published


- I wonder if a bunch of the components where you run aclocal, autoconf,
automake, etc. could be reduced to autoreconf, but would have to go look up
  the differences before pushing for it instead of just wondering.

I did experiment with this for a couple of the components as I was trying to fix them up, but was unsuccessful. This is certainly worth revisiting, especially
when we also have the new version of libtool integrated into Userland
(CR #6936997) in a future build.

Thanks for the review (and you too Craig).


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

Reply via email to