Date: Sat, 07 Nov 2020 16:02:49 +0000 From: bugzilla-noreply@freebsd.org To: ports-bugs@FreeBSD.org Subject: [Bug 250876] devel/gindent: update port Message-ID: <bug-250876-7788-F4FGqmhMkm@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-250876-7788@https.bugs.freebsd.org/bugzilla/> References: <bug-250876-7788@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D250876 --- Comment #5 from commit-hook@FreeBSD.org --- A commit references this bug: Author: mikael Date: Sat Nov 7 16:02:43 UTC 2020 New revision: 554402 URL: https://svnweb.freebsd.org/changeset/ports/554402 Log: devel/gindent: update port Update of devel/gindent o Patch buggy patch-src_args.c. It segfaults when used with -orig, -knf, = -kr, or -gnu. The extension of the options struct in args.c by *_PRO_SETTINGS is brok= en. For p_obj a string of a group of options is given. However, in the PRO_SETTINGS case, set_option() expects an index for the settings_string array instead. Details: - Remove BSD_PRO_SETTINGS as its string matches exactly ORIG_SETTINGS_STRING. So the intended behavior of -orig is alre= ady available. - Remove KR_PRO_SETTINGS as it does not correspond to Kernighan-Ritchie style. The flags -sar and -par seem to be missing. Since they a= re included in KR_SETTINGS_STRING, use that string instead. - Remove GNU_PRO_SETTINGS as its string matches exactly GNU_SETTINGS_STRING; use it instead. - Remove KNF_PRO_SETTINGS and introduce KNF_SETTINGS_STRING and u= se it like the other settings strings. This makes a version bump necessary, we therefore set PORTREVISION=3D1. o Patch Makefile.in to make --htmldir work. Remove post-install target as it's not necessary anymore. o Change PORTVERSION to DISTVERSION (PHB Sect. 5.2.2). Modify DISTNAME accordingly. o Make COMMENT more meaningful. o Remove MAKE_JOBS_UNSAFE as tests suggest that it's not necessary (anymo= re). o Add groff:run to USES variable. This registers a dependency on textproc/groff. It's needed to render the manpage. o Added description for NLS option, which is slightly more verbose then t= he default description. PR: 250876 Submitted by: Moritz Schmitt <moritz@schmi.tt> (maintainer) Changes: head/devel/gindent/Makefile head/devel/gindent/files/patch-doc_Makefile.in head/devel/gindent/files/patch-src_args.c --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-250876-7788-F4FGqmhMkm>