Bug 191046

Summary: sysutils/moosefs-master: Upgrade to 1.6.27-5 and various fixes
Product: Ports & Packages Reporter: takeda
Component: Individual Port(s)Assignee: John Marino <marino>
Status: Closed FIXED    
Severity: Affects Only Me CC: chifeng, marino, takeda
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on: 186411    
Bug Blocks:    
Attachments:
Description Flags
moosefs-master
none
moosefs-chunkserver
none
moosefs-client
none
moosefs-master none

Description takeda 2014-06-15 02:00:35 UTC
Created attachment 143788 [details]
moosefs-master

This is a patch upgrades moosefs to 1.6.27-5.

It also includes following fixes:
- support for staging
- fixing issues with config locations
- updated rc scripts
- cgiserv now runs as an unprivileged user
- ability to reload config without restarting

Note: please update moosefs-chunkserver and moosefs-client at the same time.
Comment 1 takeda 2014-06-15 02:01:43 UTC
Created attachment 143789 [details]
moosefs-chunkserver
Comment 2 takeda 2014-06-15 02:02:16 UTC
Created attachment 143790 [details]
moosefs-client
Comment 3 takeda 2014-06-15 02:11:21 UTC
Created attachment 143791 [details]
moosefs-master

Minor fixes.
Comment 4 takeda 2014-06-22 03:25:45 UTC
Could I get an update regarding this?
Comment 5 chifeng 2014-06-27 02:59:42 UTC
Yes, I think we can commit these patchs. Thanks!
Comment 6 John Marino freebsd_committer freebsd_triage 2014-07-12 08:12:52 UTC
I must be missing something.  These patches remove bsd.port.moosefs.mk which defines PORTVERSION and PORTREVISION among other things, but it doesn't seem to move the definition to the master port.

All I see is DISTVERSION hardcoded.

So what gives?  Is this a giant oversight or did I miss something?
Comment 7 John Marino freebsd_committer freebsd_triage 2014-07-12 08:40:29 UTC
okay, looking at bsd.port.mk, it seems PORTVERSION is not needed if DISTVERSION is given.  I don't think that removes the need for PORTREVISION?=0 though, otherwise slaves can't define PORTREVISION independently.
Comment 8 John Marino freebsd_committer freebsd_triage 2014-07-12 08:43:22 UTC
I am going to convert all these .cfg.dist to .cfg.sample so we can use the cleaner @sample keyword in pkg-plist
Comment 9 John Marino freebsd_committer freebsd_triage 2014-07-12 09:17:27 UTC
It turns out that the vendor makefile tries to change ownership to mfs:mfs before the user and group are created, so I have to patch the makefile.  Since I'm patching it anyway, I'm putting the .dist => .sample change here as well instead of post-install.

combine with using @owner, @group in the pkg-plist, it's passing poudriere testport now.
Comment 10 John Marino freebsd_committer freebsd_triage 2014-07-12 10:06:17 UTC
okay, I see now that there is no shared information like PORTVERSION or distinfo.  Each slave port stands alone, so there's no need for portrevision?= 0
Comment 11 commit-hook freebsd_committer freebsd_triage 2014-07-12 10:17:16 UTC
A commit references this bug:

Author: marino
Date: Sat Jul 12 10:17:00 UTC 2014
New revision: 361607
URL: http://svnweb.freebsd.org/changeset/ports/361607

Log:
  sysutils/moosefs-*: Upgrade version 1.6.27 => 1.6.27-5 and Stage support

  This was a master and two slave ports.  Now each port stands alone, each
  with it's own defined distinfo which allows for the separate mk to be
  removed.

    * Ownership of slave ports restored to chifeng.  They were reset in
      error because their staging was combined with master staging
    * Configure locations uniformly fixed to /etc/mfs
    * cgiserv now runs as unpriviledged user
    * Fixes ability to reload config without restarting
    * Uses @sample keyword for config files now
    * fixes -master, -chunkserver, and -client ports

  PR:		191046
  Submitted by:	takeda (takeda.tk)
  Approved by:	maintainer (chifeng)
  Minor fixes:	marino

Changes:
  head/sysutils/moosefs-chunkserver/Makefile
  head/sysutils/moosefs-chunkserver/distinfo
  head/sysutils/moosefs-chunkserver/files/mfschunkserver.in
  head/sysutils/moosefs-chunkserver/files/patch-mfsdata_Makefile.in
  head/sysutils/moosefs-chunkserver/files/pkg-deinstall.in
  head/sysutils/moosefs-chunkserver/pkg-plist
  head/sysutils/moosefs-client/Makefile
  head/sysutils/moosefs-client/distinfo
  head/sysutils/moosefs-client/files/
  head/sysutils/moosefs-client/files/patch-mfsdata_Makefile.in
  head/sysutils/moosefs-client/pkg-plist
  head/sysutils/moosefs-master/Makefile
  head/sysutils/moosefs-master/bsd.port.moosefs.mk
  head/sysutils/moosefs-master/distinfo
  head/sysutils/moosefs-master/files/mfscgiserv.in
  head/sysutils/moosefs-master/files/mfsmaster.in
  head/sysutils/moosefs-master/files/mfsmetalogger.in
  head/sysutils/moosefs-master/files/patch-mfsdata_Makefile.in
  head/sysutils/moosefs-master/files/pkg-deinstall.in
  head/sysutils/moosefs-master/files/pkg-install.in
  head/sysutils/moosefs-master/pkg-plist
Comment 12 John Marino freebsd_committer freebsd_triage 2014-07-12 10:20:49 UTC
Okay, it's done.  chifeng has been removed as maintainer from the slave ports but I put him back.  He was removed since no PR existed for his ports but obviously it did (just combined with -master PR)