Heads-up: rofi-pass is now packaged on FreeBSD

Question

Heads-up: rofi-pass is now packaged on FreeBSD

Opened this issue 6 years ago · 7 comments

0mp commented 6 years ago

Here's a link for more details: https://www.freshports.org/x11/rofi-pass/

Cheers!

Answer 1 · 2019-02-28T10:47:17.000Z

Were there any issues without GNU grep and awk?

Both are listed as dependencies (see README.md, when we probably could do without.

Answer 2 · 2019-02-28T12:41:27.000Z

I am not sure, really. I've not tested in personally; I just wanted to share the good news.

I'll let the maintainer about this thread we have here so that they could chime in.

Answer 3 · 2019-02-28T12:45:59.000Z

Well I added them to the dependencies because you list them in README.md. I will do some testing and report my finding back to you.

Answer 4 · 2019-02-28T13:33:27.000Z

@xanderio I like shipping portable scripts, and rofi-pass doesn't do any sort of dark magic AFAICT, so it should be feasible to rely only on POSIX options that FreeBSD grep and awk do implement (possibly even making an OpenBSD port possible in the process).

Answer 5 · 2019-03-04T19:06:57.000Z

I did some testing and it appears the GNU grep/awk are not needed for rofi-pass to work.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=236225

Answer 6 · 2019-03-05T08:44:05.000Z

Might be worth reflecting those in README.md and making sure future contributions use POSIX options for grep and awk. (I remember doing something similar for openRA)

Answer 7 · 2019-03-05T10:33:33.000Z

fyi:

Hi, I did a pass over this to make sure it wasn't using grep features that we don't quite support but don't treat as errors -- that turned out alright, but it turns out the sed usage is slightly wrong in one place [1] for our world. I would recommend sending a patch upstream to change this to the more portable expression `sed -Ee 's/[[:blank:]]+$//'`, though it's probably not critical to patch it locally since it just strips whitespace off the end (or doesn't, as the case is now =)). Thanks, Kyle Evans

[1] https://github.com/carnager/rofi-pass/blob/master/addpass#L56

…

On Tue, 05. Mar 00:44, Moviuro wrote: Might be worth reflecting those in README.md and making sure future contributions use POSIX options for grep and awk. (I remember doing something similar for openRA) — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.*