Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 12 Sep 2014 03:10:38 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   [Bug 193542] [MAINTAINER] x11-fonts/alef: Bump PORTREVISION
Message-ID:  <bug-193542-13-0xKpQAbx6B@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-193542-13@https.bugs.freebsd.org/bugzilla/>
References:  <bug-193542-13@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=193542

Jason E. Hale <jhale@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Triage                |In Discussion
                 CC|                            |jhale@FreeBSD.org

--- Comment #2 from Jason E. Hale <jhale@FreeBSD.org> ---
- Comment should not start with indefinite article "A"
- Installation commands should not be muted (mkdirs OK)
- Installation of DOCS should not be conditional.  Staging will take care of it
automatically
- To install the DOCS without the for loop, the easiest way is:
  ${INSTALL_DATA} ${PORTDOCS:S|^|${WRKDIR}/|} ${STAGEDIR}${DOCSDIR}
- This port has a BUILD_DEPENDS on fc-cache(1), which it doesn't even use
- It should be running mkfontdir(1) and mkfontscale(1) in the directory where
the fonts are.  Of course, the generated files need added to pkg-plist.
- It would be better if NO_WRKSUBDIR were set, then you wouldn't have to
redefine WRKSRC and it would eliminate the first two lines of the post-extract
target.
- Since LICENSE_FILE is being added, it should be removed from PORTDOCS so that
it isn't installed twice

-- 
You are receiving this mail because:
You are the assignee for the bug.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-193542-13-0xKpQAbx6B>