Bug 253099 - x11-fonts/paratype: update to the PT Public Pack
Summary: x11-fonts/paratype: update to the PT Public Pack
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2021-01-30 16:09 UTC by Greg V
Modified: 2021-02-19 17:20 UTC (History)
1 user (show)

See Also:
greg: maintainer-feedback+


Attachments
pt-public-pack.patch (5.46 KB, patch)
2021-01-30 16:09 UTC, Greg V
greg: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Greg V 2021-01-30 16:09:03 UTC
Created attachment 222025 [details]
pt-public-pack.patch

The PT Public Pack https://www.paratype.com/collections/pt/44157 includes new typefaces PT Astra Sans/Serif and PT Root UI, and has reasonable file names instead of whatever weird abbreviations were there in the previous downloads.

Unfortunately the download requires a Referer header, looks like this is the first use of `FETCH_ENV+= HTTP_REFERER=…` in the ports tree. Also there's no version number anywhere for the collection, so I went ahead with the download ID.
Comment 1 Li-Wen Hsu freebsd_committer 2021-02-13 12:57:45 UTC
I am worried that the download ID, which is used as PORTVERSION here, might changed or even go backward. For this situation I suggest we use a customized version or snapshot date. What do you think? Also I suggest appending a meaningful filename suffix for the distfile.
Comment 2 Greg V 2021-02-19 17:20:30 UTC
(In reply to Li-Wen Hsu from comment #1)
Okay, date is fine.

> I suggest appending a meaningful filename suffix for the distfile

How do you rename downloaded files in the ports framework? I haven't noticed any feature that allows doing this yet…