Skip to content
This repository has been archived by the owner on May 16, 2024. It is now read-only.

Source fonts naming #3898

Closed
wants to merge 3 commits into from
Closed

Conversation

heavywatal
Copy link

This PR addresses the issue #3867.
The following changes have been made.

  • Source Sans 3: new cask
  • Source Sans Pro: downgrade to version 2
  • Source Serif 4: new cask
  • Source Serif Pro: update file source and point to version 3

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

After making all changes to a cask, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused.
  • Checked the cask is submitted to the correct repo.
  • brew audit --new-cask {{cask_file}} worked successfully.
  • brew install --cask {{cask_file}} worked successfully.
  • brew uninstall --cask {{cask_file}} worked successfully.

This PR addresses the issue Homebrew#3867.
The following changes have been made.

- Source Sans 3: new cask
- Source Sans Pro: downgrade to version 2
- Source Serif 4: new cask
- Source Serif Pro: update file source and point to version 3
@Loyalsoldier
Copy link
Contributor

Loyalsoldier commented Apr 17, 2021

I think it's better to do like this:

Source Sans: new cask
Source Sans Pro: downgrade to version 2
Source Serif
Source Serif Pro: update file source and point to version 3

And use caveats to warn users about the newer release of casks like #3900

And deprecate then delete Source Sans Pro and Source Serif Pro in a year or so.

@vitorgalvao
Copy link
Member

What’s the point in keeping the old versions? They’re even failing livecheck, which should indicate the new ones are the continuation.

@@ -0,0 +1,30 @@
cask "font-source-sans-3" do
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Versions shouldn’t be part of the name.

version "3.028"
sha256 "1186d594ecb1336679a1d102bb6ddbe6041f0fd029482e53f02e46aa3ab011cf"

url "https://github.com/adobe-fonts/source-sans/releases/download/#{version}R/source-sans-#{version.sub(/\./, "v")}R.zip"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
url "https://github.com/adobe-fonts/source-sans/releases/download/#{version}R/source-sans-#{version.sub(/\./, "v")}R.zip"
url "https://github.com/adobe-fonts/source-sans/releases/download/#{version}R/source-sans-#{version.major}v#{version.minor}R.zip"

sha256 "1186d594ecb1336679a1d102bb6ddbe6041f0fd029482e53f02e46aa3ab011cf"

url "https://github.com/adobe-fonts/source-sans/releases/download/#{version}R/source-sans-#{version.sub(/\./, "v")}R.zip"
name "Source Sans 3"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No version in name.

regex(%r{tag/(\d+(?:\.\d+)*)}i)
end

font "OTF/SourceSans3-Black.otf"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
font "OTF/SourceSans3-Black.otf"
font "OTF/SourceSans#{version.major}-Black.otf"

In all of them.

@vitorgalvao vitorgalvao added the awaiting user reply Issue needs response from a user. label Apr 17, 2021
@heavywatal
Copy link
Author

Thank you for your feedback. I will consider caveats and the other requested changes after finishing this discussion.

But first, please read the upstream issues linked in #3867 again. "Source Sans 3" is not just the version 3 of "Source Sans", but it is the new family name. There is no font family named "Source Sans". Besides, the new one seems not geometrically upward-compatible with old "Source Sans Pro". In other words, silent upgrading from "Source Sans Pro" to "Source Sans 3" (and removing old "Source Sans Pro") will break existing documents. And it is expected to happen again when a new font family like "Source Sans 4" is released.

(I personally don't like their decision, but) keeping old fonts will be necessary to protect users' environments and documents. Now I have two questions:

  1. Based on the fact that "Source Sans 3" is the family name, should the cask be named source-sans-3 or source-sans( and source-sans@3 in the future)?
  2. Is it possible to bypass livecheck for casks without livecheck entry?

@vitorgalvao
Copy link
Member

"Source Sans 3" is not just the version 3 of "Source Sans", but it is the new family name.

I mean, kind of. The repo doesn’t have the number in the name, nor does the first sentence in the README.

image

(I personally don't like their decision, but)

Neither do I, but coming from Adobe I’m not surprised. We’ve removed plenty of their software from Homebrew Cask for user-hostility, and I’m inclined to do the same for these fonts.

Our current preferred way to add fonts to this repo is via Google Fonts, and I see they do have the Pro version. So it’s probably best to get rid of these and let Google Fonts deal with the naming however deem best; we’ll follow it automatically.

  1. Based on the fact that "Source Sans 3" is the family name, should the cask be named source-sans-3 or source-sans( and source-sans@3 in the future)?

We don’t do the @version for casks and definitely don’t keep different versions of fonts. They already are more trouble than they’re worth without that.

2. Is it possible to bypass livecheck for casks without livecheck entry?

CI always runs the check.

Sorry (and thank you) for the spent effort for a refusal, but it was necessary to see this in context to understand the situation. You can still use the fonts in your personal tap, though.

Will leave it open for a bit in case there are other ideas. Or perhaps they’ll come to their senses (though I doubt it).

@heavywatal
Copy link
Author

OK, agree that they should be automatically managed via Google Fonts. And now I understand that each cask name should correspond to its distribution, not fonts in it. Thank you for reviewing the issue and PR.

FYI, by "family name" I meant the attribute of the font files (how it is referenced in an OS):
Screen Shot 2021-04-18 at 0 44 17

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
awaiting user reply Issue needs response from a user.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants