Bug 198042

Summary: [MAINTAINER] security/maia: Update to 1.0.4, Lots of PERL fixes
Product: Ports & Packages Reporter: ek
Component: Individual Port(s)Assignee: Kurt Jaeger <pi>
Status: Closed FIXED    
Severity: Affects Only Me CC: perl, pi
Priority: --- Keywords: needs-qa, patch
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Diff for security/maia port.
koobs: maintainer-approval+
New patch for maia-1.0.4.
none
Latest diff.
none
Poudriere log. none

Description ek 2015-02-25 19:06:19 UTC
Created attachment 153533 [details]
Diff for security/maia port.
Comment 1 ek 2015-02-25 19:07:06 UTC
I submitted the same update a little over a week ago using 'send-pr' but I understand that 'send-pr' is no longer working. So, I'm sorry if this is a duplicate.
Comment 2 Kurt Jaeger freebsd_committer freebsd_triage 2015-02-28 11:02:37 UTC
The staging does not work because the chgrp happens before the
group is created:

install -m 640 /usr/local/home/pi/myp/security/maia/work/maia-1.0.4/maia.conf.dist /usr/local/home/pi/myp/security/maia/work/stage/usr/local/etc/maia/maia.conf.dist
/usr/bin/chgrp vscan /usr/local/home/pi/myp/security/maia/work/stage/usr/local/etc/maia/maia*
chgrp: vscan: illegal group name

Can you check this issue ?
Comment 3 Mathieu Arnold freebsd_committer freebsd_triage 2015-02-28 11:27:39 UTC
(In reply to Kurt Jaeger from comment #2)

You're partially right, the chgrp fails here because as root, the group does not exist, but it's more because it can't work, staging must work as a regular user, and you can't chgrp then.
Comment 4 ek 2015-02-28 19:54:20 UTC
Ah. This makes sense. I'm not sure how I missed this before... Any suggestions on best practice to fix it? I'd like to keep the permissions the same. Perhaps a chmod/chown instead?
Comment 5 Kurt Jaeger freebsd_committer freebsd_triage 2015-02-28 20:38:09 UTC
There are several places in the porters handbook that hint on how this is done using the pkg-plist file:

http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/book.html#plist-keywords

which refers to this example:

@sample(games,games,640) etc/config.sample

and

7.6.7.6. @owner user

which sets the default ownership for all subsequent files to user. Use without an argument to set back to default ownership (root).

(similar for @group etc).
Comment 6 ek 2015-02-28 20:46:07 UTC
Gotcha'. I'm actually already using this technique throughout my pkg-plist file. Perhaps that's how I'd missed it before (maybe I removed it for some reason..?) I'll fix this and re-submit a new patch after I test the staging process.
Comment 7 ek 2015-02-28 21:25:04 UTC
Created attachment 153629 [details]
New patch for maia-1.0.4.

Fixed ${CHGRP} issue in Makefile and added missing files in pkg-plist.
Comment 8 ek 2015-02-28 21:26:06 UTC
Thanks for your help, guys. Everything appears to install correctly now.
Comment 9 Kurt Jaeger freebsd_committer freebsd_triage 2015-03-01 16:07:38 UTC
When I try to build it, with 

DEVELOPER=yes

in /etc/make.conf, the pkg-plist gives many errors. It does not build in
poudriere.
Comment 10 Kurt Jaeger freebsd_committer freebsd_triage 2015-03-01 16:08:02 UTC
I build it with

make check-plist
Comment 11 Kurt Jaeger freebsd_committer freebsd_triage 2015-03-01 18:23:34 UTC
build logs:

http://people.freebsd.org/~pi/logs/security__maia*
Comment 12 ek 2015-03-05 01:10:15 UTC
Interesting... Thanks for the info Kurt. I'll take a look at this and get it all fixed.
Comment 13 ek 2015-03-29 05:52:40 UTC
Created attachment 154949 [details]
Latest diff.

Latest 1.0.4 patch.
Comment 14 ek 2015-03-29 05:53:20 UTC
Created attachment 154950 [details]
Poudriere log.

Poudriere success log.
Comment 15 ek 2015-03-29 05:54:23 UTC
Hi Guys,

Looks like I've finally got this all sorted out (sorry it took so long). I've attached the patch for the port as well as the successful Poudriere log. Everything seems to be working as expected. If you have any other issues, please let me know.
Comment 16 Kurt Jaeger freebsd_committer freebsd_triage 2015-03-29 07:29:38 UTC
testing@work
Comment 17 commit-hook freebsd_committer freebsd_triage 2015-03-29 08:42:13 UTC
A commit references this bug:

Author: pi
Date: Sun Mar 29 08:41:55 UTC 2015
New revision: 382571
URL: https://svnweb.freebsd.org/changeset/ports/382571

Log:
  security/maia: 1.0.3 -> 1.0.4

  TODO: find a suitable changelog

  PR:		198042
  Submitted by:	ek@purplehat.org (maintainer)

Changes:
  head/security/maia/Makefile
  head/security/maia/distinfo
  head/security/maia/files/pkg-message.in
  head/security/maia/pkg-plist
Comment 18 Kurt Jaeger freebsd_committer freebsd_triage 2015-03-29 08:43:17 UTC
Committed, thanks for the hard work!

If possible, can you provide a ChangeLog -- what has changed since the
last version ?