[Bug 611175] Review Request: yash - Yet Another SHell
bugzilla at redhat.com
bugzilla at redhat.com
Sun Jul 4 11:49:49 UTC 2010
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=611175
--- Comment #5 from Iain Arnell <iarnell at gmail.com> 2010-07-04 07:49:48 EDT ---
Thanks Ralf.
(In reply to comment #4)
> (In reply to comment #3)
> > MUSTFIX:
> > * Missing /etc/shells entry
>
> - Will add scriptlets at %post{,un} (borrowing what zsh does
> on %post{,un} - Well, with this yash will depend on bash on
> scriptlets, however I guess bash is installed on system anyway,
> and I don't know about lua....)
Agreed - the zsh scriptlets are good to copy.
> > Consider:
> > * abandon the *-doc package and bundle the doc with the main package.
> > This separate doc-package makes not much sense
>
> - Well, it seems that size of yash{-doc} is (much) smaller
> than bash/zsh, will merge.
I actually liked that the (no offence intended, but useless to most people who
don't understand Japanese) docs were completely separate. I know there's some
way to exclude languages, but I can't for the life of me remember how. And if
the docs grow to include translations in future, they will likely become bigger
than the main rpm.
> > * Trim the description.
> - Will remove.
Agreed.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the package-review
mailing list