Bug 224113

Summary: devel/gogs: Switch to "git" user, move the configuration files to /usr/local/etc
Product: Ports & Packages Reporter: Dmitri Goutnik <dmgk>
Component: Individual Port(s)Assignee: Yuri Victorovich <yuri>
Status: Closed FIXED    
Severity: Affects Some People CC: douglas, felix_mail, sascha.folie, yuri
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://lists.freebsd.org/pipermail/freebsd-ports/2017-December/111400.html
Attachments:
Description Flags
patch file
dmgk: maintainer-approval+
New location for app.ini
none
patch file
dmgk: maintainer-approval+
patch file
dmgk: maintainer-approval+
patch file
dmgk: maintainer-approval+
patch file dmgk: maintainer-approval+

Description Dmitri Goutnik freebsd_committer freebsd_triage 2017-12-05 14:35:47 UTC
Created attachment 188551 [details]
patch file

* Switch Gogs user from "gogs" to "git"
* While here, change RUN_MODE from "dev" to "prod"
* Bump PORTREVISION
* Add UPDATING entry

QA:
  portlint -AC: OK
  poudriere testport: OK (11a, 10i)
Comment 1 Felix Z. 2017-12-06 11:27:41 UTC
Hi Dmitri.
Can you move the configuration files to /usr/local/etc ?
Comment 2 Felix Z. 2017-12-06 11:28:00 UTC
Created attachment 188581 [details]
New location for app.ini

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=224113#c1
Comment 3 Dmitri Goutnik freebsd_committer freebsd_triage 2017-12-06 16:40:44 UTC
Created attachment 188587 [details]
patch file

Hi Felix,

Thank you for your patch, I've incorporated it with some changes.
Comment 4 Dmitri Goutnik freebsd_committer freebsd_triage 2017-12-06 16:43:26 UTC
Created attachment 188588 [details]
patch file
Comment 5 Dmitri Goutnik freebsd_committer freebsd_triage 2017-12-06 17:31:37 UTC
Created attachment 188590 [details]
patch file

Fix portlint warning.
Comment 6 Felix Z. 2017-12-07 05:14:39 UTC
(In reply to Dmitri Goutnik from comment #3)

Dmitri, thanks!
Comment 7 Yuri Victorovich freebsd_committer freebsd_triage 2017-12-12 23:51:53 UTC
https://reviews.freebsd.org/D13469
Comment 8 Dmitri Goutnik freebsd_committer freebsd_triage 2017-12-13 22:37:05 UTC
Created attachment 188814 [details]
patch file

Make "Log path" input read-only on the Install page.
Based on the latest https://reviews.freebsd.org/D13469
Comment 9 commit-hook freebsd_committer freebsd_triage 2017-12-18 23:57:14 UTC
A commit references this bug:

Author: yuri
Date: Mon Dec 18 23:57:07 UTC 2017
New revision: 456695
URL: https://svnweb.freebsd.org/changeset/ports/456695

Log:
  devel/gogs: Changed user from gogs to git; Moved ini file under /usr/local/etc

  PR:		224113
  Submitted by:	Dmitri Goutnik <dg@syrec.org> (maintainer)
  Approved by:	adamw (mentor)
  Differential Revision:	https://reviews.freebsd.org/D13469

Changes:
  head/GIDs
  head/UIDs
  head/UPDATING
  head/devel/gogs/Makefile
  head/devel/gogs/files/gogs-rm-data.in
  head/devel/gogs/files/gogs-service.in
  head/devel/gogs/files/gogs.in
  head/devel/gogs/files/patch-templates_install.tmpl
  head/devel/gogs/files/pkg-message.in
  head/devel/gogs/pkg-plist
Comment 10 Yuri Victorovich freebsd_committer freebsd_triage 2017-12-18 23:57:19 UTC
Committed, thanks!