Missing comma in fonts.mss may hide Armenian font

[Sjord]

fonts.mss contains this:

@book-fonts:    "Noto Sans Regular",
                "Noto Sans CJK JP Regular",
                "Noto Sans Adlam Regular", "Noto Sans Adlam Unjoined Regular"
                "Noto Sans Armenian Regular",
                "Noto Sans Balinese Regular",

There is no comma between Noto Sans Adlam Unjoined Regular and Noto Sans Armenian Regular. The result seems to be that Noto Sans Armenian Regular disappears from the created mapnik.xml:

  <FontSet name="fontset-0">
    <Font face-name="Noto Sans Regular" />
    <Font face-name="Noto Sans CJK JP Regular" />
    <Font face-name="Noto Sans Adlam Regular" />
    <Font face-name="Noto Sans Adlam Unjoined Regular" />
    <Font face-name="Noto Sans Balinese Regular" />

After adding a comma after "Noto Sans Adlam Unjoined Regular", the Armenian fonts appear in the mapnik XML again.

I am not affected by this issue, and I am not familiar with Armenian mapping fonts. It was just something I noticed while working with these MSS files.

[jeisenbe]

Thank you for catching this bug, @Sjord - would you have time to submit a pull request to fix this bug?

Here is a screenshot of Armenia at the moment:

z7

z8

Compare with this sample of https://fontinfo.opensuse.org/fonts/NotoSansArmenianRegular.html

I believe the style has been falling back to Unifont for rendering Armenian characters, but it would be good to fix this bug so that the Armenian characters match the usual style of the Noto font family which is normally used.

[Sjord]

would you have time to submit a pull request to fix this bug?

Yes, I can make a pull request somewhere this week.