xlsx2csv @1.2 darwin: bad bash test for gnused searching exact file gsed without path and never to be found

[energo80]

This is current code in line

gnused=$(which sed)
[[ "${platform}" == "Darwin" ]] && gnused=gsed
[[ -f "$gnused" ]] || { ERROR....}

should be changed to

which $gnused || error

[ryandesign (Ryan Carsten Schmidt)]

You should not rely on which returning a nonzero exit code when a program is not found; on Tiger, it doesn't do that.

I propose a different fix; attached.

[kurthindenburg (Kurt Hindenburg)]

Kirk's site doesn't appear to be valid any longer - all I could find is ​https://web.archive.org/web/20150910223552/http://kirk.webfinish.com/?p=91

Perhaps it should be removed and another actively maintainer converter could be added.

Also, on the couple of xlsx files I have, the script didn't work

[bithead2k]

Sorry, website was created with wordpress. It got hacked, and i never got it fixed.

I didn't see the notification for this bug until now. This port is not very active. The patch looks reasonable, so I'll integrate it.

[mf2k (Frank Schima)]

@bithead2k: ping?

[kurthindenburg (Kurt Hindenburg)]

In c3a3b5ae74684f301519475d24101278338d06d0/macports-ports:

xlsx2csv: add patch to find gsed

set homepage to web.archive.org link; set livecheck to none

closes #52954