On 03/ 7/12 01:22 PM, Craig Mohrman wrote:
62                 /usr/gnu/bin/sed -i \
      We have a $GSED macro for that.

64                         -e 's|^CC="/.*$$|CC="cc"|' libtool ; \
                                       ^ drop
      You can drop that "/".

Actually I can't. There are other 'CC=""' entries further down the
libtool files which I don't want to change.

Should match the LTCC pattern directly above this one.
      Doing the kind of search.

Not exactly.

Thanks for the review.




----- rich.burri...@oracle.com wrote:

Hi all,

Could I please get a code review for:

    7101897 libtool contains some automount paths
    http://monaco.us.oracle.com/detail.jsf?cr=7101897

Webrev is at:

    http://jurassic.sfbay/~richb/7101897-v1/

Userland workspace (with just libtool built) is at:

    /net/stard.us.oracle.com/tank/ws/UL/7101897/

    Build/publish log is at:


/net/stard.us.oracle.com/tank/ws/UL/7101897/components/libtool/publish-trans.txt

See the test related Comment in the Bugster CR for how I tested this.

(Thanks to alanc for help with this).

_______________________________________________
userland-discuss mailing list
userland-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/userland-discuss

_______________________________________________
userland-discuss mailing list
userland-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/userland-discuss

Reply via email to