Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

50-user.conf fontconfig file no longer generated #95712

Closed
pbogdan opened this issue Aug 17, 2020 · 1 comment · Fixed by #95716
Closed

50-user.conf fontconfig file no longer generated #95712

pbogdan opened this issue Aug 17, 2020 · 1 comment · Fixed by #95716
Labels
0.kind: bug Something is broken

Comments

@pbogdan
Copy link
Member

pbogdan commented Aug 17, 2020

Describe the bug

When fonts.fontconfig.includeUserConf is set 50-user.conf file should be placed somewhere under /etc/fonts.

Expected behavior

I expect 50-user.conf to exist under /etc/fonts.

Screenshots

N/A

Additional context

N/A

Notify maintainers

Metadata

nixpkgs revision: 16fc531

Maintainer information:

# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:
@pbogdan pbogdan added the 0.kind: bug Something is broken label Aug 17, 2020
This was referenced Aug 17, 2020
Closed
Closed
flokli added a commit to flokli/nixpkgs that referenced this issue Aug 17, 2020
@flokli
nixos/fontconfig: fix 50-user.conf handling
8425726 
Apparently, edf2541 was missed while
rebasing NixOS#93562.

Provide 50-user.conf in fontconfig if includeUserConf is true (the
default), and don't try removing the non-existent one if it's disabled

Fixes NixOS#95685
Fixes NixOS#95712
@flokli flokli mentioned this issue Aug 17, 2020
Merged
10 tasks
@flokli
Copy link
Contributor

flokli commented Aug 17, 2020

Should be fixed by #95716.

wchresta pushed a commit to wchresta/nixpkgs that referenced this issue Aug 17, 2020
@flokli @wchresta
nixos/fontconfig: fix 50-user.conf handling
494c887 
Apparently, edf2541 was missed while
rebasing NixOS#93562.

Provide 50-user.conf in fontconfig if includeUserConf is true (the
default), and don't try removing the non-existent one if it's disabled

Fixes NixOS#95685
Fixes NixOS#95712
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0.kind: bug Something is broken
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

nixos/fontconfig: fix 50-user.conf handling flokli/nixpkgs
2 participants
@pbogdan @flokli