Request 1007813 (accepted)

You're not reviewing the full diff of request 1007813 , but the diff to the superseded request 1007204 (Show full diff)

- Replace tcsh-6.24.01-bash52.dif with upstream fix (forwarded request 1007809 from favogt)

Why not use the patch I submitted upstream? https://github.com/tcsh-org/tcsh/pull/53

author source maintainer

As I know how bash now works due to the changes for the lastpipe shell option caused by an idea from mine https://lists.gnu.org/archive/html/bug-bash/2010-06/msg00097.html. Now the support of lastpipe seems to be more robust

I don't get the connection to the question here...

author source maintainer

Avoid to become command last in a pipe/subshell ... that is what ;: does. Without overriding the environment.

I know what your patch does, but why not take the one I submitted upstream?

author source maintainer

I'd like to avoid env(1) as it manipulates the enviroment

But that's exactly the point here

- Replace tcsh-6.24.01-bash52.dif with upstream fix (forwarded request 1007809 from favogt)

factory-auto added opensuse-review-team as a reviewer about 2 years ago

Please review sources

Check script succeeded

favogt_factory set openSUSE:Factory:Staging:A as a staging project about 2 years ago

Being evaluated by staging project "openSUSE:Factory:Staging:A"

Picked "openSUSE:Factory:Staging:A"

Staging Project openSUSE:Factory:Staging:A got accepted.

Staging Project openSUSE:Factory:Staging:A got accepted.

Staging Project openSUSE:Factory:Staging:A got accepted.