On 2023/01/24 20:08, Omar Polo wrote:
> On 2023/01/24 18:37:55 +0000, Job Snijders <j...@openbsd.org> wrote:
> > Hi all,
> >
> > Attached is a port of textproc/grepcidr3, this fork of grepcidr has a
> > few command line options that the original lacks, such as -D:
> >
> > $ echo 10.1.1.0/24 | grepcidr3 -D 10.1.1.1
> > 10.1.1.0/24
> >
> > I added 2 patches:
> > * rename the utility to 'grepcidr3' (in order to avoid naming collisions)
> > * add pledge("stdio rpath", NULL) in main().
> >
> > OK?
>
> there's a CVS directory still left in pkg/
>
> needs a little tweak to respect CFLAGS and it's also custom to add a
> comment marker for stuff using pledge.
>
> --- Makefile.orig Tue Jan 24 19:55:47 2023
> +++ Makefile Tue Jan 24 19:56:16 2023
> @@ -11,7 +11,10 @@
> # BSD 2-Clause and GPLv2+
> PERMIT_PACKAGE= Yes
>
> +# uses pledge()
> WANTLIB += c
> +
> +MAKE_FLAGS += CFLAGS="${CFLAGS}"
>
> NO_TEST= Yes
>
>
> could also set the version to something like 0.0pl20230107 since
> upstream hasn't tagged any version, even if it says 'grepcidr 3.0'
> with -V. would avoid setting EPOCH if upstream starts tagging with
> something lower than 3.0.
>
> otherwise the port looks OK for me.
+1 from me
> Don't know much about the history of the tool, there's any specific
> reason to not just upgrade the existing textproc/grepcidr to this
> fork instead?
This does some things that grepcidr doesn't do, and grepcidr does some
things that this doesn't do.
