Date: Mon, 1 Apr 2024 04:17:09 +0800 From: Po-Chuan Hsieh <sunpoet@freebsd.org> To: =?UTF-8?Q?T=C4=B3l_Coosemans?= <tijl@freebsd.org> Cc: "Tobias C. Berner" <tcberner@gmail.com>, Gleb Popov <arrowd@freebsd.org>, ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 28472e9722d8 - main - Revert "x11-fonts/noto*: Restructure and update" Message-ID: <CAMHz58QZ0bQh51s4j9CNt7yRZX8=uQi-cDp3ok6FN8ie0v=Gbw@mail.gmail.com> In-Reply-To: <20240324202715.5d18838d@hal.tijl.coosemans.org> References: <202403231404.42NE4baU095633@gitrepo.freebsd.org> <CALH631=Q0Hc%2BYnvREbe0aNR%2BVeWaNv3YdekKmeUe_Kf6tXD6TQ@mail.gmail.com> <2567403.0dHE6SNnxz@angua.firefly> <CAMHz58Qa2pyN6qs-mFr9aHgtQoGrd1hbhF=ahx5ahR-0CB-7uQ@mail.gmail.com> <20240324202715.5d18838d@hal.tijl.coosemans.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000f1c6cc0614fa9002 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, Mar 25, 2024 at 3:27=E2=80=AFAM T=C4=B3l Coosemans <tijl@freebsd.or= g> wrote: > On Sun, 24 Mar 2024 00:12:07 +0800 Po-Chuan Hsieh wrote: > > On Sat, Mar 23, 2024 at 11:38=E2=80=AFPM Tobias C. Berner <tcberner@gma= il.com> > > wrote: > >> Am Samstag, 23. M=C3=A4rz 2024, 15:14:01 CET schrieb Gleb Popov: > >>> On Sat, Mar 23, 2024 at 5:04=E2=80=AFPM Po-Chuan Hsieh <sunpoet@freeb= sd.org> > >>> wrote: > >>>> The branch main has been updated by sunpoet: > >>>> > >>>> URL: > >>>> > https://cgit.FreeBSD.org/ports/commit/?id=3D28472e9722d808739a4d690f6d803= a2b1b8f5f98 > >>>> > >>>> commit 28472e9722d808739a4d690f6d803a2b1b8f5f98 > >>>> Author: Po-Chuan Hsieh <sunpoet@FreeBSD.org> > >>>> AuthorDate: 2024-03-23 14:01:07 +0000 > >>>> Commit: Po-Chuan Hsieh <sunpoet@FreeBSD.org> > >>>> CommitDate: 2024-03-23 14:01:07 +0000 > >>>> > >>>> Revert "x11-fonts/noto*: Restructure and update" > >>>> > >>>> This reverts commit 3b128ff9f74b28ada2f475f0b816480b1550e5b8. > >>> > >>> It is good manners to explain the reason for reverting when doing it. > >>> Even better to first talk to the original committer and ask him to th= e > >>> revert. > >> > >> Moin moin > >> > >> I completely agree. Given that there has been a phabricator review ope= n > >> for two months, with you, sunpoet@ as reviewer, this is looking like > poor > >> judgement on your side. > >> > >> As arrowd@ points out, you should have commented on the reverted commi= t > >> first, and elaborate your reasoning for a revert. > >> > >> Simply reverting the work of others may lead to fewer contributions. > >> Please take this into consideration in the future. > >> > >> > >> mfg Tobias > > > > > > Hello, > > > > I'm sorry that I did not put enough information in this reverting commi= t > > because I need it to be done ASAP so that less users are affected. > > There are several reasons I reverted it by myself instead of > > requesting tijl@ to do so. > > First, I do not use phab review therefore I did not really review the > patch > > which means it is committed without maintainer approval. > > Second, I already have my own work for noto updates which is totally in > > different direction with tijl@'s work. I cannot simply rebase my work > onto > > it. > > Third, The problem mentioned in the PR is also handled in my own work. > > > > Regards, > > sunpoet > > I don't mind getting reverted. I do think Noto is large enough that > some thought should go into how we handle it in the ports tree and that > you should have put your alternative up for discussion before committing > it. Here are some (possible) issues with it that should probably be > fixed before the next quarterly: > > - You are using upstream distfiles that are quite large. The distfiles > for the common (Latin, Greek, Cyrillic) fonts Sans, Sans Mono, and > Serif add up to 234MB. These fonts will be installed by almost every > user that uses FreeBSD as a desktop now that DejaVu appears to have > been abandoned. My proposal was to create our own distfiles and I > integrated that into makesum so the only extra step compared to a > normal version update was to upload the generated distfile to > freefall. I was willing to take over maintainership in case you > didn't want to deal with that. These distfiles add up to 27MB. And > if we put the extra weights (ExtraBold,...) and widths (Condensed,...) > in a separate port like you did it would be only 2MB. > The committed one is my second version of noto fonts update. The previous version is to use the individual file from github.com just like those Noto CJK ports. I changed to upstream tarball because it's clearer/simpler and easier to track. I could change it back to the previous version for a smaller distfile footprint. Give me several days to clean up the Makefiles. > - Some fonts have been split into x11-fonts/<font>-basic providing Bold, > BoldItalic, Italic, and Regular, and x11-fonts/<font>-extra providing > additional styles. Only some of these have x11-fonts/<font> as a > metaport currently. This needs to be made consistent. I think it > would be best to drop the metaports and move x11-fonts/<font>-basic to > x11-fonts/<font>. That way pkg install <font> installs the common > styles for all fonts. I think that this would be the most > intuitive/convenient. > (I was hoping to use subpackages for this instead of extra ports.) > The -basic/-extra structure was kept for minimal change. I agree with you that it would be better to be consistent with all other noto ports. I could merge the following ports: - noto-sans-basic/-extra - noto-sans-mono-basic/-extra - noto-sans-symbols-basic/-extra - noto-serif-basic/-extra However, what should be noto-basic's dependencies after the merge? > - x11/noto-emoji now installs the COLRv1 version of Noto Color Emoji but > this isn't supported yet by graphics/cairo so emoji are broken now in > gtk applications. I think Ghostscript doesn't support it either. It > has to be removed again for now. > It is handled in PR 278019 and ports 2878ca490586207c3cd7bb3ff94d1274f52013b0. > > - Besides Noto Color Emoji there's also the monochrome Noto Emoji that > can be useful in print. I think we should let x11-fonts/noto-emoji > install the monochrome font and move the existing x11-fonts/noto-emoji > to x11-fonts/noto-color-emoji so the names match. > How about putting all emoji in the existing port (x11-fonts/noto-emoji)? --000000000000f1c6cc0614fa9002 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr">On Mon, Mar 25, 2024 at 3:27=E2=80=AFAM T= =C4=B3l Coosemans <<a href=3D"mailto:tijl@freebsd.org" target=3D"_blank"= >tijl@freebsd.org</a>> wrote:<br></div><div class=3D"gmail_quote"><block= quote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1= px solid rgb(204,204,204);padding-left:1ex">On Sun, 24 Mar 2024 00:12:07 +0= 800 Po-Chuan Hsieh wrote:<br> > On Sat, Mar 23, 2024 at 11:38=E2=80=AFPM Tobias C. Berner <<a href= =3D"mailto:tcberner@gmail.com" target=3D"_blank">tcberner@gmail.com</a>>= <br> > wrote:<br> >> Am Samstag, 23. M=C3=A4rz 2024, 15:14:01 CET schrieb Gleb Popov:= =C2=A0 <br> >>> On Sat, Mar 23, 2024 at 5:04=E2=80=AFPM Po-Chuan Hsieh <<a = href=3D"mailto:sunpoet@freebsd.org" target=3D"_blank">sunpoet@freebsd.org</= a>>=C2=A0 <br> >>> wrote:=C2=A0 <br> >>>> The branch main has been updated by sunpoet:<br> >>>><br> >>>> URL:<br> >>>> <a href=3D"https://cgit.FreeBSD.org/ports/commit/?id=3D284= 72e9722d808739a4d690f6d803a2b1b8f5f98" rel=3D"noreferrer" target=3D"_blank"= >https://cgit.FreeBSD.org/ports/commit/?id=3D28472e9722d808739a4d690f6d803a= 2b1b8f5f98</a><br> >>>><br> >>>> commit 28472e9722d808739a4d690f6d803a2b1b8f5f98<br> >>>> Author:=C2=A0 =C2=A0 =C2=A0Po-Chuan Hsieh <sunpoet@Free= BSD.org><br> >>>> AuthorDate: 2024-03-23 14:01:07 +0000<br> >>>> Commit:=C2=A0 =C2=A0 =C2=A0Po-Chuan Hsieh <sunpoet@Free= BSD.org><br> >>>> CommitDate: 2024-03-23 14:01:07 +0000<br> >>>><br> >>>>=C2=A0 =C2=A0 =C2=A0Revert "x11-fonts/noto*: Restructu= re and update"<br> >>>><br> >>>>=C2=A0 =C2=A0 =C2=A0This reverts commit 3b128ff9f74b28ada2f= 475f0b816480b1550e5b8.=C2=A0 <br> >>><br> >>> It is good manners to explain the reason for reverting when do= ing it.<br> >>> Even better to first talk to the original committer and ask hi= m to the<br> >>> revert.=C2=A0 <br> >><br> >> Moin moin<br> >><br> >> I completely agree. Given that there has been a phabricator review= open<br> >> for two months, with you, sunpoet@ as reviewer, this is looking li= ke poor<br> >> judgement on your side.<br> >><br> >> As arrowd@ points out, you should have commented on the reverted c= ommit<br> >> first, and elaborate your reasoning for a revert.<br> >><br> >> Simply reverting the work of others may lead to fewer contribution= s.<br> >> Please take this into consideration in the future.<br> >><br> >><br> >> mfg Tobias=C2=A0 <br> > <br> > <br> > Hello,<br> > <br> > I'm sorry that I did not put enough information in this reverting = commit<br> > because I need it to be done ASAP so that less users are affected.<br> > There are several reasons I reverted it by myself instead of<br> > requesting tijl@ to do so.<br> > First, I do not use phab review therefore I did not really review the = patch<br> > which means it is committed without maintainer approval.<br> > Second, I already have my own work for noto updates which is totally i= n<br> > different direction with tijl@'s work. I cannot simply rebase my w= ork onto<br> > it.<br> > Third, The problem mentioned in the PR is also handled in my own work.= <br> > <br> > Regards,<br> > sunpoet<br> <br> I don't mind getting reverted.=C2=A0 I do think Noto is large enough th= at<br> some thought should go into how we handle it in the ports tree and that<br> you should have put your alternative up for discussion before committing<br= > it.=C2=A0 Here are some (possible) issues with it that should probably be<b= r> fixed before the next quarterly:<br> <br> - You are using upstream distfiles that are quite large.=C2=A0 The distfile= s<br> =C2=A0 for the common (Latin, Greek, Cyrillic) fonts Sans, Sans Mono, and<b= r> =C2=A0 Serif add up to 234MB.=C2=A0 These fonts will be installed by almost= every<br> =C2=A0 user that uses FreeBSD as a desktop now that DejaVu appears to have<= br> =C2=A0 been abandoned.=C2=A0 My proposal was to create our own distfiles an= d I<br> =C2=A0 integrated that into makesum so the only extra step compared to a<br= > =C2=A0 normal version update was to upload the generated distfile to<br> =C2=A0 freefall.=C2=A0 I was willing to take over maintainership in case yo= u<br> =C2=A0 didn't want to deal with that.=C2=A0 These distfiles add up to 2= 7MB.=C2=A0 And<br> =C2=A0 if we put the extra weights (ExtraBold,...) and widths (Condensed,..= .)<br> =C2=A0 in a separate port like you did it would be only 2MB.<br></blockquot= e><div><br></div><div>The committed one is my second version of noto fonts = update.</div><div>The previous version is to use the individual file from <= a href=3D"http://github.com" target=3D"_blank">github.com</a> just like tho= se Noto CJK ports.</div><div>I changed to upstream tarball because it's= clearer/simpler and easier to track.</div><div>I could change it back to t= he previous version for a smaller distfile footprint.</div><div>Give me sev= eral days to clean up the Makefiles.</div><div><br></div><blockquote class= =3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rg= b(204,204,204);padding-left:1ex"> <br> - Some fonts have been split into x11-fonts/<font>-basic providing Bo= ld,<br> =C2=A0 BoldItalic, Italic, and Regular, and x11-fonts/<font>-extra pr= oviding<br> =C2=A0 additional styles.=C2=A0 Only some of these have x11-fonts/<font&= gt; as a<br> =C2=A0 metaport currently.=C2=A0 This needs to be made consistent.=C2=A0 I = think it<br> =C2=A0 would be best to drop the metaports and move x11-fonts/<font>-= basic to<br> =C2=A0 x11-fonts/<font>.=C2=A0 That way pkg install <font> inst= alls the common<br> =C2=A0 styles for all fonts.=C2=A0 I think that this would be the most<br> =C2=A0 intuitive/convenient.<br> =C2=A0 (I was hoping to use subpackages for this instead of extra ports.)<b= r></blockquote><div><br></div><div>The=C2=A0-basic/-extra structure was kep= t for minimal change.</div><div>I agree with you that it would be better to= be consistent with all other noto ports.<br></div><div><div>I could merge = the following ports:</div></div><div>-=C2=A0noto-sans-basic/-extra</div><di= v>-=C2=A0noto-sans-mono-basic/-extra</div><div>-=C2=A0noto-sans-symbols-bas= ic/-extra</div><div>-=C2=A0noto-serif-basic/-extra</div><div>However, what = should be noto-basic's dependencies after the merge?=C2=A0</div><div><b= r></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex= ;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br> - x11/noto-emoji now installs the COLRv1 version of Noto Color Emoji but<br= > =C2=A0 this isn't supported yet by graphics/cairo so emoji are broken n= ow in<br> =C2=A0 gtk applications.=C2=A0 I think Ghostscript doesn't support it e= ither.=C2=A0 It<br> =C2=A0 has to be removed again for now.<br></blockquote><div><br></div><div= >It is handled in PR 278019 and ports=C2=A02878ca490586207c3cd7bb3ff94d1274= f52013b0.</div><div>=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"= margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-lef= t:1ex"> <br> - Besides Noto Color Emoji there's also the monochrome Noto Emoji that<= br> =C2=A0 can be useful in print.=C2=A0 I think we should let x11-fonts/noto-e= moji<br> =C2=A0 install the monochrome font and move the existing x11-fonts/noto-emo= ji<br> =C2=A0 to x11-fonts/noto-color-emoji so the names match.<br></blockquote><d= iv><br></div><div>How about putting all emoji in the existing port (x11-fon= ts/noto-emoji)?</div></div></div> --000000000000f1c6cc0614fa9002--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAMHz58QZ0bQh51s4j9CNt7yRZX8=uQi-cDp3ok6FN8ie0v=Gbw>