Bug 254747 - [NEW PORT] devel/rgxg regular expression generator (cli util and lib)
Summary: [NEW PORT] devel/rgxg regular expression generator (cli util and lib)
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
Depends on:
Reported: 2021-04-03 19:56 UTC by alexander naumochkin
Modified: 2021-04-18 18:02 UTC (History)
1 user (show)

See Also:

devel/rgxg new port (1.76 KB, application/x-shellscript)
2021-04-03 19:56 UTC, alexander naumochkin
no flags Details
devel/rgxg new port (1.78 KB, application/x-shellscript)
2021-04-04 20:50 UTC, alexander naumochkin
no flags Details
Patch for rgxg (2.12 KB, patch)
2021-04-05 21:33 UTC, daniel.engberg.lists
no flags Details | Diff
devel/rgxg new port (1.83 KB, application/x-shellscript)
2021-04-10 07:31 UTC, alexander naumochkin
no flags Details
Patch for rgxg v2 (2.03 KB, patch)
2021-04-18 18:00 UTC, daniel.engberg.lists
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description alexander naumochkin 2021-04-03 19:56:22 UTC
Created attachment 223790 [details]
devel/rgxg new port

extended regular expression generator especially useful for ip (v4 and v6) matching regexes
Comment 1 daniel.engberg.lists 2021-04-04 05:56:27 UTC
Hi Alexander,

A few things that can be improved. :-)

Porters Handbook - "Table 5.2. Package Naming Examples"

"If the distribution file comes from a specific commit or tag on GitHub for which there is no officially released file"
In this case upstream provides one, https://github.com/rgxg/rgxg/releases/tag/v0.1.2 --> https://github.com/rgxg/rgxg/releases/download/v0.1.2/rgxg-0.1.2.tar.gz so please use that instead.

Please define DOCS and possibly MANPAGES as port options

Thanks for your contribution!
Best regards,
Comment 2 alexander naumochkin 2021-04-04 20:50:11 UTC
Created attachment 223810 [details]
devel/rgxg new port

updated Makefile for port
Comment 3 alexander naumochkin 2021-04-04 20:57:49 UTC
Hi Daniel,

I've modified Makefile following your recomendations about *VERSION and MASTER_SITES, check it, please.

But at the moment I'm not sure what to do with DOCS/MANPAGES without patching configure.ac, because docs/man-pages installation are not options with it. Do I have to make the patch or ask author for that?

Sorry for my newbie questions, this is my first port :)
Comment 4 daniel.engberg.lists 2021-04-05 21:33:38 UTC
Created attachment 223845 [details]
Patch for rgxg

Add pkgconf(ig) as dependency
Disable static library
Add DOCS option
Comment 5 daniel.engberg.lists 2021-04-05 21:35:35 UTC
I initially didn't look at upstream but as you said upstream doesn't have options for it. The ports framework does however let you do create these options as it can automatically ignore file(s) if you "bind" them to an option ("knob") (OPTIONS_SUB in Porters Handbook, PORTDOCS enables it by default and adds items automatically to plist) however there's no need to overengineer so I just made a DOCS option.

I should also mention that I learned a few new things thanks to jrm@ and mat@ by hacking on this. :-)

This version also passes Poudriere on 12.2-RELEASE

Best regards,
Comment 6 alexander naumochkin 2021-04-10 07:31:39 UTC
Created attachment 223976 [details]
devel/rgxg new port

Corrected (and final I hope) version
Comment 7 alexander naumochkin 2021-04-10 07:40:33 UTC
Hi Daniel,

thanks alot for your mentoring, I've attached new edition of .shar. Hope it is good enough to adopt. If everything is Ok, should I do something else with this "bug-report"?

Best regards,
Comment 8 daniel.engberg.lists 2021-04-18 18:00:21 UTC
Created attachment 224227 [details]
Patch for rgxg v2

Remove "Created by" and deprecated SVN entry ( $FreeBSD$ )

Poudriere testport OK 12.2-RELEASE (amd64)
Poudriere testport OK 11.4-RELEASE (amd64)
Comment 9 daniel.engberg.lists 2021-04-18 18:02:30 UTC
Hi Alexander,

Can you please review "Patch for rgxg v2" I've done some style cleanup and COMMENT= follows ports framework style (no non standard capitals)

This should be ready to commit

Best regards,