Bug 206798 - sysutils/desktop-installer: Add Lumina desktop support
Summary: sysutils/desktop-installer: Add Lumina desktop support
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Raphael Kubo da Costa
URL:
Keywords: easy, patch, patch-ready
Depends on: 206797
Blocks:
  Show dependency treegraph
 
Reported: 2016-01-31 14:17 UTC by Jason W. Bacon
Modified: 2016-02-17 12:25 UTC (History)
1 user (show)

See Also:
koobs: maintainer-feedback+


Attachments
Unified diff (1.84 KB, text/plain)
2016-01-31 14:17 UTC, Jason W. Bacon
jwb: maintainer-approval+
Details
Poudriere log (10.90 KB, text/x-log)
2016-02-03 04:00 UTC, Jason W. Bacon
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jason W. Bacon freebsd_committer freebsd_triage 2016-01-31 14:17:49 UTC
Created attachment 166355 [details]
Unified diff

Add a configuration option for Lumina desktop.

Add lagg interface option for wifi users.

Make fdesc mount mandatory in case bash or openjdk installed later.

Improve removable media mount config for XFCE.

Other minor enhancements and bug fixes.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2016-01-31 14:17:49 UTC
Maintainer informed via mail
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2016-01-31 15:46:39 UTC
Thanks Jason

Could you also:

* Confirm this change passes QA (poudriere, portlint)
* Update your bugzilla account email to match the port MAINTAINER address so you can set maintainer approval on the attachment

Thanks!
Comment 3 Jason W. Bacon freebsd_committer freebsd_triage 2016-01-31 19:26:33 UTC
I don't have a working Poudriere installation at the moment, but I run portlint -A immediately before submitting every PR and I have DEVELOPER=yes on every machine I use for development.

I'm hoping to get Poudriere working shortly and intend to attach a log to all future PRs.

I'm stumped on the email issue.  I changed it the day 204755 was committed, but it apparently didn't stick.  I just tried to change it again.
Comment 4 Chris Hutchinson 2016-01-31 20:49:56 UTC
(In reply to Jason Bacon from comment #3)
> I don't have a working Poudriere installation at the moment, but I run
> portlint -A immediately before submitting every PR and I have DEVELOPER=yes
> on every machine I use for development.
> 
> I'm hoping to get Poudriere working shortly and intend to attach a log to
> all future PRs.
> 
> I'm stumped on the email issue.  I changed it the day 204755 was committed,
> but it apparently didn't stick.  I just tried to change it again.

Jason, FWIW there are also a couple of other options available;
Having DEVELOPER=yes also allows you to *test* the entire process
from anywhere on your system. For example, I do everything in
~/DEV
for example, to test changes to sysutils/jailutils
mkdir -p ~/DEV/sysutils/jailutils/SVN
cd ~/DEV/sysutils/jailutils/SVN
$ svn co --depth empty svn://svn.freebsd.org/ports/head 2016-01-31
$ svn up --set-depth empty 2016-01-31/sysutils
$ svn up 2016-01-27/sysutils/jailutils

EDIT files in 2016-01-27/sysutils/jailutils
then perform
$ script ~/DEV/sysutils-jailutils-QA
$ make -DBATCH check-plist
$ make stage-qa
$ make check-sanity
$ portlint -AC
$ uname -a
$ exit

The only side-affect doing this, is that the ports framework
will complain that it won't check category, because it knows
it's not working within a complete ports tree.

Another option that works quite nicely, and is dead simple
to setup, is ports-mgmt/synth
It produces QA LOGS very similar to those produced by Poudriere

--Chris
Comment 5 Jason W. Bacon freebsd_committer freebsd_triage 2016-02-03 03:11:49 UTC
Thanks for the tips!  I got my poudriere installation working, though.  It wasn't that hard - just needed to find the time to debug it.  I also scripted a poudriere setup wizard to easily create a typical jail for ports testing.

And BTW, I forgot to mention that this update depends on PR 206797 in case anyone didn't catch that.
Comment 6 Jason W. Bacon freebsd_committer freebsd_triage 2016-02-03 04:00:26 UTC
Created attachment 166475 [details]
Poudriere log
Comment 7 Kubilay Kocak freebsd_committer freebsd_triage 2016-02-04 07:11:39 UTC
Comment on attachment 166475 [details]
Poudriere log

No need for approvals on logs
Comment 8 commit-hook freebsd_committer freebsd_triage 2016-02-17 12:25:33 UTC
A commit references this bug:

Author: rakuco
Date: Wed Feb 17 12:25:10 UTC 2016
New revision: 409041
URL: https://svnweb.freebsd.org/changeset/ports/409041

Log:
  Update to 0.5.0.

  - Add a configuration option for Lumina desktop.
  - Add lagg interface option for wifi users.
  - Make fdesc mount mandatory in case bash or openjdk installed later.
  - Improve removable media mount config for XFCE.
  - Other minor enhancements and bug fixes.

  PR:		206798
  Submitted by:	Jason Bacon <bacon4000@gmail.com> (maintainer)

Changes:
  head/sysutils/desktop-installer/Makefile
  head/sysutils/desktop-installer/distinfo
  head/sysutils/desktop-installer/pkg-plist
Comment 9 Raphael Kubo da Costa freebsd_committer freebsd_triage 2016-02-17 12:25:53 UTC
Committed, thanks!