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
diff --git a/debian/71-ubuntulegacy.conf b/debian/71-ubuntulegacy.conf
0new file mode 1006440new file mode 100644
index 0000000..7f2a5e6
--- /dev/null
+++ b/debian/71-ubuntulegacy.conf
@@ -0,0 +1,20 @@
1<?xml version='1.0'?>
2<!DOCTYPE fontconfig SYSTEM 'fonts.dtd'>
3<selectfont>
4 <rejectfont>
5 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-B.ttf</glob>
6 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-BI.ttf</glob>
7 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-C.ttf</glob>
8 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-L.ttf</glob>
9 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-LI.ttf</glob>
10 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-M.ttf</glob>
11 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-MI.ttf</glob>
12 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-R.ttf</glob>
13 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-RI.ttf</glob>
14 <glob>/usr/share/fonts/truetype/ubuntu/Ubuntu-Th.ttf</glob>
15 <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-B.ttf</glob>
16 <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-BI.ttf</glob>
17 <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-R.ttf</glob>
18 <glob>/usr/share/fonts/truetype/ubuntu/UbuntuMono-RI.ttf</glob>
19 </rejectfont>
20</selectfont>
diff --git a/debian/fonts-ubuntu.install b/debian/fonts-ubuntu.install
index d0e353a..3163350 100644
--- a/debian/fonts-ubuntu.install
+++ b/debian/fonts-ubuntu.install
@@ -1 +1,2 @@
1Ubuntu*.ttf usr/share/fonts/truetype/ubuntu1Ubuntu*.ttf usr/share/fonts/truetype/ubuntu
2debian/71-ubuntulegacy.conf usr/share/fontconfig/conf.avail
diff --git a/debian/fonts-ubuntu.links b/debian/fonts-ubuntu.links
index f399c34..fc5d96b 100644
--- a/debian/fonts-ubuntu.links
+++ b/debian/fonts-ubuntu.links
@@ -12,4 +12,5 @@ usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/u
12usr/share/fonts/truetype/ubuntu/Ubuntu-Italic[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-RI.ttf12usr/share/fonts/truetype/ubuntu/Ubuntu-Italic[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-RI.ttf
13usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-R.ttf13usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-R.ttf
14usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-Th.ttf14usr/share/fonts/truetype/ubuntu/Ubuntu[wdth,wght].ttf usr/share/fonts/truetype/ubuntu/Ubuntu-Th.ttf
15usr/share/fontconfig/conf.avail/71-ubuntulegacy.conf etc/fonts/conf.d/71-ubuntulegacy.conf
1516

Subscribers

People subscribed via source and target branches