Differential D14001

x11-themes/sddm-freebsd-black-theme: improving the port.
ClosedPublic
Actions

Authored by rigoletto on Jan 20 2018, 8:28 PM.

Tags

None

Referenced Files

	F139623005: D14001.id39063.diff
	Sun, Dec 14, 6:56 AM

	F139604188: D14001.id54425.diff
	Sun, Dec 14, 1:17 AM

	Unknown Object (File)
	Thu, Dec 11, 5:07 AM

	Unknown Object (File)
	Mon, Dec 1, 8:49 AM

	Unknown Object (File)
	Fri, Nov 21, 7:22 AM

	Unknown Object (File)
	Fri, Nov 21, 7:17 AM

	Unknown Object (File)
	Fri, Nov 21, 7:15 AM

	Unknown Object (File)
	Fri, Nov 21, 7:12 AM

Subscribers

Contributor Reviewers (ports)

Details

Reviewers

adridg
decke

Commits

rP461336: - Add missing dependency on montserrat font

Summary

switch to DISTVERSION
add an OPTION of the default font

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rigoletto created this revision.Jan 20 2018, 8:28 PM

Harbormaster completed remote builds in B14505: Diff 38259.Jan 20 2018, 8:28 PM

Herald added subscribers: Contributor Reviewers (ports), mat. · View Herald TranscriptJan 20 2018, 8:28 PM

rigoletto added a reviewer: adridg.Jan 20 2018, 10:07 PM

adridg added inline comments.Feb 8 2018, 3:32 PM

x11-themes/sddm-freebsd-black-theme/Makefile
24 ↗	(On Diff #38259)	Does this need to be an option? Why not make it required? Same applies to your other port, I don't understand why a font (presumably required to make the theme look "right") is an optional dependency.
27 ↗	(On Diff #38259)	Indentation issues? (Sorry, hard to tell in phab's web interface)

adridg mentioned this in D14002: x11-themes/slim-freebsd-black-theme: New Port..Feb 8 2018, 3:33 PM

decke accepted this revision.Feb 8 2018, 4:00 PM

decke added a subscriber: decke.

This comment was removed by decke.

This revision is now accepted and ready to land.Feb 8 2018, 4:00 PM

Sorry that was the wrong Phabricator ticket. It was meant for D14002.

I'd propose the following patch:

D14001-1.diff582 BDownload

Done. :)

This revision now requires review to proceed.Feb 8 2018, 8:26 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 9 2018, 8:29 PM

Closed by commit rP461336: - Add missing dependency on montserrat font (authored by decke). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

x11-themes/

sddm-freebsd-black-theme/

5 lines

Diff 39110

head/x11-themes/sddm-freebsd-black-theme/Makefile

Loading...