Skip to content

Conversation

@pmenzel
Copy link
Collaborator

As discussed in the Teatime, and commented in #2819 (comment).

@pmenzel Paul Menzel (pmenzel) changed the title fonts-indic: Put all Indic fonts (Lohit) in one bee file Put all Indic fonts (Lohit) in one bee file Jan 6, 2023
@pmenzel Paul Menzel (pmenzel) force-pushed the put-all-indico-fonts-in-one-bee-file branch from f3def5b to d2771c6 Compare January 6, 2023 11:14
@donald
Copy link
Collaborator

Donald Buczek (donald) commented Jan 6, 2023 •

Should the individual files be removed in the same PR?

"Remove the individual font packages with..." would not qualify as the right imperative for a code change to commit-message purist :-)

LGTM

This basically removes merg/pull request #2819 from
mariux64/add-indic-fonts.

On the system, you can remove the individual font packages with

    $ for p in $(fakeroot bee list lohit); do sudo bee remove $p; done

resulting in

    sudo bee remove lohit-assamese-ttf-2.91.5-0.x86_64
    sudo bee remove lohit-bengali-ttf-2.91.5-0.x86_64
    sudo bee remove lohit-devanagari-ttf-2.95.4-0.x86_64
    sudo bee remove lohit-gujarati-ttf-2.92.4-0.x86_64
    sudo bee remove lohit-gurmukhi-ttf-2.91.2-0.x86_64
    sudo bee remove lohit-kannada-ttf-2.5.4-0.x86_64
    sudo bee remove lohit-malayalam-ttf-2.92.2-0.x86_64
    sudo bee remove lohit-marathi-ttf-2.94.2-0.x86_64
    sudo bee remove lohit-nepali-ttf-2.94.2-0.x86_64
    sudo bee remove lohit-odia-ttf-2.91.2-0.x86_64
    sudo bee remove lohit-tamil-classical-ttf-2.5.4-0.x86_64
    sudo bee remove lohit-tamil-ttf-2.91.3-0.x86_64
    sudo bee remove lohit-telugu-ttf-2.5.5-0.x86_64
@pmenzel Paul Menzel (pmenzel) force-pushed the put-all-indico-fonts-in-one-bee-file branch from d2771c6 to 72b9980 Compare January 6, 2023 17:28
@pmenzel
Copy link
Collaborator Author

Yes, the individual bee files should be removed. Sorry for missing that.

@donald
Copy link
Collaborator

Just a note, that the list of installed files differs. I don't know why or of that is important.

buczek@theinternet:~/git/bee-files (put-all-indico-fonts-in-one-bee-file)$ diff  x.1.s x.2.s
2a3,5
> /etc/fonts/conf.d/59-lohit-devanagari.conf///usr/share/fontconfig/conf.avail/59-lohit-devanagari.conf
> /etc/fonts/conf.d/65-0-lohit-marathi.conf///usr/share/fontconfig/conf.avail/65-0-lohit-marathi.conf
> /etc/fonts/conf.d/65-0-lohit-nepali.conf///usr/share/fontconfig/conf.avail/65-0-lohit-nepali.conf
5d7
< /etc/fonts/conf.d/66-lohit-devanagari.conf///usr/share/fontconfig/conf.avail/66-lohit-devanagari.conf
9,11d10
< /etc/fonts/conf.d/66-lohit-malayalam.conf///usr/share/fontconfig/conf.avail/66-lohit-malayalam.conf
< /etc/fonts/conf.d/66-lohit-marathi.conf///usr/share/fontconfig/conf.avail/66-lohit-marathi.conf
< /etc/fonts/conf.d/66-lohit-nepali.conf///usr/share/fontconfig/conf.avail/66-lohit-nepali.conf
15a15
> /etc/fonts/conf.d/67-lohit-malayalam.conf///usr/share/fontconfig/conf.avail/67-lohit-malayalam.conf
17a18,31
> /usr/share/fontconfig/conf.avail/59-lohit-devanagari.conf
> /usr/share/fontconfig/conf.avail/65-0-lohit-marathi.conf
> /usr/share/fontconfig/conf.avail/65-0-lohit-nepali.conf
> /usr/share/fontconfig/conf.avail/66-lohit-assamese.conf
> /usr/share/fontconfig/conf.avail/66-lohit-bengali.conf
> /usr/share/fontconfig/conf.avail/66-lohit-devanagari.conf
> /usr/share/fontconfig/conf.avail/66-lohit-gujarati.conf
> /usr/share/fontconfig/conf.avail/66-lohit-gurmukhi.conf
> /usr/share/fontconfig/conf.avail/66-lohit-kannada.conf
> /usr/share/fontconfig/conf.avail/66-lohit-odia.conf
> /usr/share/fontconfig/conf.avail/66-lohit-tamil-classical.conf
> /usr/share/fontconfig/conf.avail/66-lohit-tamil.conf
> /usr/share/fontconfig/conf.avail/66-lohit-telugu.conf
> /usr/share/fontconfig/conf.avail/67-lohit-malayalam.conf

@pmenzel
Copy link
Collaborator Author

Yes, I only mentioned that in the commit message:

This also actually installs the *conf files, and not just the symbolic links.

This also actually installs the *conf files, and not just the symbolic
links.
@pmenzel Paul Menzel (pmenzel) force-pushed the put-all-indico-fonts-in-one-bee-file branch from 72b9980 to a0bad97 Compare January 8, 2023 07:48
@pmenzel Paul Menzel (pmenzel) merged commit 4818904 into master Jan 9, 2023
@pmenzel
Copy link
Collaborator Author

(NB: Typo in branch name indico ;-))

Sign in to join this conversation on GitHub.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants