Merge ~seb128/ubuntu/+source/fonts-ubuntu:ignore-compat-symlinks into ubuntu/+source/fonts-ubuntu:ubuntu/devel

Proposed by Sebastien Bacher
Status: Merged
Merge reported by: Sebastien Bacher
Merged at revision: fb763909d6971da36e1b76c93655ed604bd75746
Proposed branch: ~seb128/ubuntu/+source/fonts-ubuntu:ignore-compat-symlinks
Merge into: ubuntu/+source/fonts-ubuntu:ubuntu/devel
Diff against target: 42 lines (+22/-0)
3 files modified
debian/71-ubuntulegacy.conf (+20/-0)
debian/fonts-ubuntu.install (+1/-0)
debian/fonts-ubuntu.links (+1/-0)
Reviewer Review Type Date Requested Status
Ubuntu Sponsors Pending
git-ubuntu import Pending
Review via email: mp+440582@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

A nitpicking thing: Would be good if it could be installed in /usr/share/fontconfig/conf.avail instead of /etc/fonts/conf.avail .

Revision history for this message
Iain Lane (laney) wrote :

Looks good (and agreed with Gunnar). I did leave one question comment inline.

If you wanted I think you could simplify the glob expression if you installed the new fonts in a different directory, then you could `rejectfont` all of `/usr/share/fonts/truetype/ubuntu/*` in one go.

81fabd8... by Sebastien Bacher

remove invalid entry

fb76390... by Sebastien Bacher

install the fontconfig entry to /usr instead of /etc

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for the review, I pushed/uploaded with the config installed to /usr and removing the weird empty tag.

> If you wanted I think you could simplify the glob expression if you installed the new fonts in a different directory

I ignored that because moving the font would create the same upgrade issue than made us add the symlinks and do another round of hacks, it didn't feel like that was worth it

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/71-ubuntulegacy.conf b/debian/71-ubuntulegacy.conf
2new file mode 100644
3index 0000000..7f2a5e6
4--- /dev/null
5+++ b/debian/71-ubuntulegacy.conf
6@@ -0,0 +1,20 @@
7+<?xml version='1.0'?>
8+<!DOCTYPE fontconfig SYSTEM 'fonts.dtd'>
9+<selectfont>
10+ <rejectfont>
11+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-B.ttf</glob>
12+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-BI.ttf</glob>
13+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-C.ttf</glob>
14+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-L.ttf</glob>
15+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-LI.ttf</glob>
16+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-M.ttf</glob>
17+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-MI.ttf</glob>
18+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-R.ttf</glob>
19+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-RI.ttf</glob>
20+ <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-Th.ttf</glob>
21+ <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-B.ttf</glob>
22+ <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-BI.ttf</glob>
23+ <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-R.ttf</glob>
24+ <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-RI.ttf</glob>
25+ </rejectfont>
26+</selectfont>
27diff --git a/debian/fonts-ubuntu.install b/debian/fonts-ubuntu.install
28index d0e353a..3163350 100644
29--- a/debian/fonts-ubuntu.install
30+++ b/debian/fonts-ubuntu.install
31@@ -1 +1,2 @@
32 Ubuntu*.ttf usr/share/fonts/truetype/ubuntu
33+debian/71-ubuntulegacy.conf usr/share/fontconfig/conf.avail
34diff --git a/debian/fonts-ubuntu.links b/debian/fonts-ubuntu.links
35index f399c34..fc5d96b 100644
36--- a/debian/fonts-ubuntu.links
37+++ b/debian/fonts-ubuntu.links
38@@ -12,4 +12,5 @@ usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/u
39 usr/share/fonts/truetype/ubuntu/Ubuntu-Italic[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-RI.ttf
40 usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-R.ttf
41 usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-Th.ttf
42+usr/share/fontconfig/conf.avail/71-ubuntulegacy.conf etc/fonts/conf.d/71-ubuntulegacy.conf
43

Subscribers

People subscribed via source and target branches