Based on feedback from Ali, it was found that my previous version of
this fix didn't work correctly.
Danek provided the solution (many thanks), and I've reworked the
changes for this fix to use his approach (see the Comment in the
Bugster CR for more details).
New webrev is at:
http://jurassic.us.oracle.com/~richb/7092165-v2/
x86 Workspace is at:
/net/stard.us.oracle.com/tank/ws/UL/7092165/
(The configure/build/install/publish transcript is in a file called
'publish-trans.txt' in the texinfo component directory).
Thanks.
-------- Original Message --------
Subject: Re: [userland-discuss] Code review request for CR #7092165
Date: Mon, 31 Oct 2011 10:44:43 -0700
From: Rich Burridge <rich.burri...@oracle.com>
To: david.co...@oracle.com
CC: Userland-Discuss <userland-discuss@opensolaris.org>
On 10/26/11 04:14 PM, Rich Burridge wrote:
On 10/26/2011 04:09 PM, david.co...@oracle.com wrote:
I still think the ls -l route is best -- it's worked for years for exactly
this problem.
+1
Okay, I'll rework it to use that.
I've revised the changes to fix:
7092165 text/texinfo's missing a dependency on file/gnu-coreutils
http://monaco.us.oracle.com/detail.jsf?cr=7092165
to use a "home grown" equivalent of 'readlink -f' in the SMF service file.
Shamelessly cribbed from one of the versions mentioned at:
http://stackoverflow.com/questions/5756524/get-absolute-path-name-of-shell-script-without-readlink-f
New webrev is at:
http://jurassic.us.oracle.com/~richb/7092165-v1/
Could I please get a +1 or a LGTM from one or more of you.
x86 Workspace (with just the component in question built) is at:
/net/stard.us.oracle.com/tank/ws/UL/7092165/
(The latest configure/build/install/publish transcript is in a file called
'publish-trans.txt' in the texinfo component directory).
See the Bugster CR for more details (including how I tested this change).
Thanks.
_______________________________________________
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
