[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