Keith Mitchell wrote:
Nit: Would it be better to list the full hierarchical names for all?
(Or am I thinking too much about a README bug...)
- Keith
On 03/31/10 07:21 AM, Luis de Bethencourt wrote:
Please review the fixes for:
Bug 14959 <http://defect.opensolaris.org/bz/show_bug.cgi?id=14959> -
README: Step 3 (required IPS packages) - pkg names should be updated
The Bug:
http://defect.opensolaris.org/bz/show_bug.cgi?id=14959
The Webrev:
http://cr.opensolaris.org/~luisbg/14959/
It's a small one but keeping the README up to date will make the life
easier of all new developers to the project.
Thanks,
Luis
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
Sorry we took this off-line. Here is a carbon copy for transparency:
<luisbg> "Would it be better to list the full hierarchical names for all?:
<luisbg> s/:/"
<luisbg> what do you mean?
<kemitche> e.g., on line 40 SUNWzoneint becomes system/zones/internal
<kemitche> Should line 42 not be "text/gnu-gettext"?
<kemitche> and 41 be "developer/swig"?
<luisbg> let me check one thing very quickly
<kemitche> 34 -> "developer/sunstudioexpress"; 37 -> "developer/build/onbld"
<luisbg> kemitche, I see your point... or we do it that way, or we do
switch "system/zones/internal" to just "internal"
<luisbg> for consistency purposes
<luisbg> so... simplicity or information?
<kemitche> "internal" is too non-informative (and I feel documentation
should generally use the full pkg names anyway, to guarantee uniqueness)
<luisbg> yes, that is why I put the full name
<luisbg> internal sounds just too ambiguos
<kemitche> Otherwise if another "foo/bar/internal" gets added down the
road - then what?
<luisbg> it's a bad name, gnu-gettext and swig are names of things...
but internal? :S
<luisbg> so I read your vote is for informative? change all to
hierarchal names?
<kemitche> Yes, my vote is informative
I also vote for informative. I will go ahead and fix the webrev. Anybody
against it?
Luis
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
