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.

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

Windows Compat name comes out as "NF" and "NF Mono" #1054

Closed
Finii opened this issue Jan 18, 2023 · 16 comments
Closed

Windows Compat name comes out as "NF" and "NF Mono" #1054

Finii opened this issue Jan 18, 2023 · 16 comments

Comments

@Finii
Copy link
Collaborator

Finii commented Jan 18, 2023

@Finii I just updated to 2.3.0 NF fonts and Mononoki's mono font has been renamed from "Mononoki NFM" to "Mononoki NF Mono". Not sure why this was necessary when the other NF fonts on my system still use "NFM" in their name, or perhaps this was a mistake? The past two times NF's Mononoki has updated I've had to fiddle with application font settings to get things working again.

Originally posted by @redactedscribe in #575 (comment)

This is a bug.

It should be one of

  • Nerd Font
  • Nerd Font Mono
  • NF
  • NFM

Maybe I put the fix into #1028, lets check

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

image

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

image

image

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

Affected: All --makegroups fonts.

i.e.

  • Cascadia Code
  • Iosevka
  • JetBrains Mono
  • mononiki

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

Fixed via cherry-picking ee02072 (as f240e07)

@Finii Finii closed this as completed Jan 18, 2023
@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

Fonts come out correct now.
I will do a bugfix release later, have to fetch my kids now, srry

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

@redactedscribe How did you update? First I thought via Casks, but the you would not have Windows Compat fonts...

I guess (hope) we do not need to release v2.3.1 but can live with a patched release 😬

@redactedscribe
Copy link

redactedscribe commented Jan 18, 2023

I used Scoop. A patched release means just redownloading* the same 2.3.0 which is now fixed?

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

I have right now no clue how Scoop works 😬

I'm sitting in the car, patcher is running, will upload fixed fonts afterwards via mobile data which can take time.
Then the fonts are at least correct in the repo.

Afterwards I want to re-package the zip files, but that is even more data and so that will probably not happen while I'm on the go but in two hours or so. Then I can inject the new zip archives into the v2.3.0 release, so that at least it looks smoothly afterwards.

I hope not many packagers already did update.
Except Cask, which I updated, but that I can fix. The casks check the zip file sha256sums...

@redactedscribe
Copy link

For GitHub releases, Scoop checks the releases page every so often for a version number greater than it last saw. Fixing the archive will likely make Scoop complain that the checksums no longer match. If it's just for a handful of fonts, it should be easy enough to put in some PRs for the scoop-nerd-fonts bucket. Maybe @jonz94 knows more.

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

Reasons for the issue:

I intended to also put #1028 into the v2.3.0 release. I checked all the resulting fonts.
But shortly before release I though #1028 should wait until v3.0.0 as this renames things which should be a major release.
So I backed off.

But one of the commits in that PR was vital for this release (see above).
Once mentioned I immediately recognized the commit, and found out it has not made it into the release.
Sigh. Sorry!

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

enough to put in some PRs for the scoop-nerd-fonts bucket. Maybe @jonz94 knows more.

Then we should probably just release v2.3.1 which is also not that hard and avoids any version misunderstandings.
But as the version is encoded in every font file that would mean touch ALL font files, which is ... huge.

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

Maybe a v2.3.0p1 🤔

This is what I would do at work, do not know if all packagers know such version number constructs.

@Finii
Copy link
Collaborator Author

Finii commented Jan 18, 2023

Release v2.3.1

@redactedscribe
Copy link

Thanks a bunch @Finii. Back to Mononoki NFM. Minor issue: The font versions weren't updated as far as I can tell. They all still read 2.3.0. A lot of moving parts. It happens.

@Finii
Copy link
Collaborator Author

Finii commented Jan 19, 2023

They all still read 2.3.0

That was deliberate (*). Anyhow I expect v2.3.1 to be rather short lived anyhow.
With v2.2.0 we got a v2.2.1 and v2.2.2 in short succession.
This is not really a bug but rather a glitch in the naming I'd say. But there will come real bug reports in, I'm sure, and for that we anyhow need to repatch all fonts.

Whatever ;)

===

(*) Maybe the reasoning seems flawed in retrospection, but it was along these lines:

  • I dont want a 2.3.1 but rather a 2.3.0-p1
  • I dont know if packaging systems can work with such a number, so
  • Use 2.3.1 as fancy way to say 2.3.0.p1 - as release version but not in the fonts - to trigger packagers updates
  • Internally use as much as possible of the 2.3.0 fonts to avoid
    • time consuming repatching of all font files
    • faster but error prone version editing of all font files
    • having the smallest possible commit (the repo is so large already)
  • Reuse 2.3.0's zip archives for the bugfix release
  • That fonts have version 2.3.0
  • Keep the few newly generated font files at the same version number for 'consistency'
  • /shrug ;-)

@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity (i.e. last half year) after it was closed. It helps our maintainers focus on the active issues. If you have found a problem that seems similar, please open a new issue, complete the issue template with all the details necessary to reproduce, and mention this issue as reference.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants