Created attachment 321878 [details, diff] patch to support cpio archives This is needed for example for app-shells/mksh.
Comment on attachment 321878 [details, diff] patch to support cpio archives >+ #needed as cpio always reads from stdin # Comments are like this >+ if [[ $1 = "-" ]]; then use "==", not "=", and put a space before that ";" >+ unpack_banner "stdin" >+ cpio --make-directories --extract --preserve-modification-time add a new local array like: local cmd=( cpio --make-directories --extract --preserve-modification-time ) then use it in both these places like: "${cmd[@]}" >+ cpio --make-directories --extract --preserve-modification-time <"$cpio" "${cpio}" >+ *.cpio.*|*.cpio) do you really need the *.cpio.* ? i guess you want to handle like .cpio.gz ?
Created attachment 321880 [details, diff] patch to support cpio archives (In reply to comment #1) > Comment on attachment 321878 [details, diff] [details, diff] > patch to support cpio archives > > >+ #needed as cpio always reads from stdin > > # Comments are like this > > >+ if [[ $1 = "-" ]]; then > > use "==", not "=", and put a space before that ";" > > >+ unpack_banner "stdin" > >+ cpio --make-directories --extract --preserve-modification-time > > add a new local array like: > local cmd=( cpio --make-directories --extract --preserve-modification-time > ) > > then use it in both these places like: > "${cmd[@]}" > > >+ cpio --make-directories --extract --preserve-modification-time <"$cpio" > > "${cpio}" You are kind of a syntax nazi :-) All done! > >+ *.cpio.*|*.cpio) > > do you really need the *.cpio.* ? i guess you want to handle like .cpio.gz ? Yes, app-shells/mksh uses cpio.gz.
(In reply to comment #2) LGTM; feel free to commit
(In reply to comment #3) > (In reply to comment #2) > > LGTM; feel free to commit Done.