Jim Walker writes:
> James Carlson wrote:
> >> BTW. Shouldn't this be "/usr/bin/ksh -p" ?
> >> 1 #! /bin/ksh -p
> >
> > /bin is (has always been) a symlink to /usr/bin. I don't think that
> > makes a big difference.
> >
> > Looking at the other scripts in this directory, I see 9 that use
> > /us
Jim Walker writes:
> Looks good.
Thanks.
> BTW. Shouldn't this be "/usr/bin/ksh -p" ?
> 1 #! /bin/ksh -p
/bin is (has always been) a symlink to /usr/bin. I don't think that
makes a big difference.
Looking at the other scripts in this directory, I see 9 that use
/usr/bin, and 28 that use /bin.
Sorry for this; I somehow missed this in my which_scm testing. It's a
very simple and obvious one-liner.
http://bugs.grommit.com/show_bug.cgi?id=505
http://cr.opensolaris.org/~carlsonj/webrev-550/
--
James Carlson, Solaris Networking
Sun Microsystems / 35 Network Drive
Richard Lowe writes:
>
> Please review: 473 webrev -p doesn't work with mercurial
>
> http://cr.opensolaris.org/~richlowe/scm_473
>
> http://bugs.grommit.com/show_bug.cgi?id=473
The changes look good to me.
--
James Carlson, Solaris Networking
Sun Microsystems / 35 Networ
http://bugs.grommit.com/show_bug.cgi?id=461
Bug 461 depends on bug 473, which changed state.
Bug 473 Summary: webrev -p doesn't work with mercurial
http://bugs.grommit.com/show_bug.cgi?id=473
What|Old Value |New Value
---
Author: Richard Lowe
Repository: /hg/scm-migration/onnv-scm
Latest revision: b1978228feabd17ec765af93988345cd63ac99a5
Total changesets: 1
Log message:
473 webrev -p doesn't work with mercurial
Files:
update: usr/src/tools/scripts/webrev.sh
James Carlson wrote:
>> BTW. Shouldn't this be "/usr/bin/ksh -p" ?
>> 1 #! /bin/ksh -p
>
> /bin is (has always been) a symlink to /usr/bin. I don't think that
> makes a big difference.
>
> Looking at the other scripts in this directory, I see 9 that use
> /usr/bin, and 28 that use /bin. So mayb
Looks good.
BTW. Shouldn't this be "/usr/bin/ksh -p" ?
1 #! /bin/ksh -p
Cheers,
Jim
James Carlson wrote:
> Sorry for this; I somehow missed this in my which_scm testing. It's a
> very simple and obvious one-liner.
>
> http://bugs.grommit.com/show_bug.cgi?id=505
>
> http://cr.opensolaris.o
http://bugs.grommit.com/show_bug.cgi?id=461
Bug 461 depends on bug 505, which changed state.
Bug 505 Summary: simple typo in flg.flp shows up under Teamware
http://bugs.grommit.com/show_bug.cgi?id=505
What|Old Value |New Value
---
Author: James Carlson
Repository: /hg/scm-migration/onnv-scm
Latest revision: 0c3452dce57f1d71b3add05fc51d62f62ab13d4c
Total changesets: 1
Log message:
505 simple typo in flg.flp shows up under Teamware
Files:
update: usr/src/tools/scripts/flg.flp.sh
> http://cr.opensolaris.org/~carlsonj/webrev-550/
Looks fine to me.
mike
http://bugs.grommit.com/show_bug.cgi?id=461
richlowe at richlowe.net changed:
What|Removed |Added
BugsThisDependsOn||505
--
Configure bugmail: http:/
I took stomper.sfbay down over the weekend because of A/C work in the
lab that stomper lives in. (Sorry, I probably should have sent mail
about this Friday.) I was able to bring stomper up briefly this
morning, but it died after a few hours with a bad root disk. I've filed
a trouble ticket, but
OK, there are now code review requests outstanding for
473 webrev -p doesn't work with mercurial
504 gpyfm evicted from onnv-scm; needs new home
...and reviewed changes awaiting putback for
476 changes for move from gpyfm
I believe that:
- Jim is reviewing 473
Rich has already gotten 476 reviewed; that's the wad that will remove
gpyfm from onnv-scm. "But," I hear you ask plaintively, "where will it
go?"
I think it should still end up installing into /opt/onbld, but via a
separate package. I chose OSOLgpyfm for that purpose.
This webrev shows only
15 matches
Mail list logo
