Bug 225786

Summary: graphics/gmt: Update to 4.5.17 and take maintainership
Product: Ports & Packages Reporter: Loïc Bartoletti <lbartoletti>
Component: Individual Port(s)Assignee: Yuri Victorovich <yuri>
Status: Closed FIXED    
Severity: Affects Some People CC: yuri
Priority: --- Keywords: easy, patch-ready
Version: LatestFlags: lbartoletti: maintainer-feedback+
Hardware: Any   
OS: Any   
Attachments:
Description Flags
gmt 4.5.17 lbartoletti: maintainer-approval+

Description Loïc Bartoletti freebsd_committer 2018-02-09 06:43:44 UTC
Created attachment 190454 [details]
gmt 4.5.17

Minor changes
Take maintainership

Poudriere 10/11 i386/amd64 OK
Porlint OK
Comment 1 Yuri Victorovich freebsd_committer 2018-02-09 21:30:19 UTC
Thank you for your patch.

In the future please try to create patches/shars from /usr/ports. It is more convenient this way.

Thank you!
Yuri
Comment 2 Yuri Victorovich freebsd_committer 2018-02-09 21:40:44 UTC
I enabled all options, and got this error:
> gmake[4]: mkoctfile: Command not found

Another problem is that it builds everything with debug flags:
> cc -pipe -fno-omit-frame-pointer  -g -fstack-protector -fno-strict-aliasing -g -Wall  -fPIC -fno-strict-aliasing -std=c99 -I/usr/ports/graphics/gmt/work/gmt-4.5.17/src -I../mgg -I/usr/local/include -I/usr/local/include  -fno-omit-frame-pointer -DUSE_GDAL -DDEBUG  -c -o x_remove.o x_remove.c

It needs to respect CFLAGS.

Another problem is that it uses has USES=autoreconf:build while 'configure' file is present. 'configure' should be preferred when present, unless it fails for some reason. In such case there should be a comment explaining this.
Comment 3 Yuri Victorovich freebsd_committer 2018-02-22 19:03:44 UTC
Committed.
Thank you for taking maintainership!
Comment 4 commit-hook freebsd_committer 2018-02-22 19:04:14 UTC
A commit references this bug:

Author: yuri
Date: Thu Feb 22 19:03:40 UTC 2018
New revision: 462620
URL: https://svnweb.freebsd.org/changeset/ports/462620

Log:
  graphics/gmt: Update to 4.5.17

  lbartoletti@tuxfamily.org takes maintainership.

  Also changed to DISTVERSION.

  PR:		225786
  Submitted by:	lbartoletti@tuxfamily.org
  Approved by:	tcberner (mentor, implicit)

Changes:
  head/graphics/gmt/Makefile
  head/graphics/gmt/distinfo