Date: Fri, 17 Jan 2025 19:11:18 +0100 From: =?UTF-8?Q?Fernando_Apestegu=C3=ADa?= <fernape@freebsd.org> To: Moin Rahman <bofh@freebsd.org> Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: a8d366898894 - main - textproc/docproj: remove gnuplot-lite dependency Message-ID: <CAGwOe2Yy77o3kLtChNoAMszKzh2ecUDJHbs4k9mizhGOuv_a2A@mail.gmail.com> In-Reply-To: <26434658-5E5B-4197-890A-2B926AA2A487@freebsd.org> References: <CAGwOe2bTFU5Z68BuZ_zy9oYfLt2UWaTLco%2B8e4q_QNuxizxojA@mail.gmail.com> <26434658-5E5B-4197-890A-2B926AA2A487@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000001d980e062bead722 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, Jan 16, 2025 at 1:21=E2=80=AFPM Moin Rahman <bofh@freebsd.org> wrot= e: > > > On 16 Jan 2025, at 13:02, Fernando Apestegu=C3=ADa <fernape@freebsd.org> = wrote: > > =EF=BB=BF > > > On Thu, Jan 16, 2025 at 8:56=E2=80=AFAM Moin Rahman <bofh@freebsd.org> wr= ote: > >> >> >> > On Jan 16, 2025, at 08:41, Fernando Apestegu=C3=ADa <fernape@FreeBSD.o= rg> >> wrote: >> > >> > The branch main has been updated by fernape: >> > >> > URL: >> https://cgit.FreeBSD.org/ports/commit/?id=3Da8d3668988945bf23844a1733da2= 14113e08d6ec >> > >> > commit a8d3668988945bf23844a1733da214113e08d6ec >> > Author: Fernando Apestegu=C3=ADa <fernape@FreeBSD.org> >> > AuthorDate: 2025-01-14 17:26:26 +0000 >> > Commit: Fernando Apestegu=C3=ADa <fernape@FreeBSD.org> >> > CommitDate: 2025-01-16 07:41:00 +0000 >> > >> > textproc/docproj: remove gnuplot-lite dependency >> > >> > It was a leftover from the previous docproj. >> > --- >> > textproc/docproj/Makefile | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/textproc/docproj/Makefile b/textproc/docproj/Makefile >> > index 6d362be1878a..a0b3c298573c 100644 >> > --- a/textproc/docproj/Makefile >> > +++ b/textproc/docproj/Makefile >> > @@ -1,5 +1,6 @@ >> > PORTNAME=3D docproj >> > PORTVERSION=3D 4.1.9 >> > +PORTREVISION=3D 1 >> > CATEGORIES=3D textproc >> > >> > MAINTAINER=3D doceng@FreeBSD.org >> > @@ -7,7 +8,6 @@ COMMENT=3D Meta-port for the FreeBSD Documentation Pro= ject >> > WWW=3D https://www.freebsd.org/docproj >> > >> > RUN_DEPENDS=3D asciidoctor:textproc/rubygem-asciidoctor \ >> > - gnuplot-lite>0:math/gnuplot-lite \ >> > gohugo>=3D0.133.1,1:www/gohugo \ >> > rougify:textproc/rubygem-rouge \ >> > rubygem-asciidoctor-diagram>0:textproc/rubygem-asciidoctor-diagram >> >> I don't think this is correct. gnuplot was used to generate the graph >> of number of ports here: >> >> https://docs.freebsd.org/en/books/dev-model/#sub-project-ports > > > Interesting. I don't see any invocations to gnuplot in any of the > Makefiles. > Also, I don't have any gnuplot* packages in my system and I regularly > build the documentation just fine: > > $ DOC_LANG=3Den make > --------------------------------------------------------------- > Building the documentation > included languages: en > excluded languages: bn-bd da de el es fr hu id it ja ko mn nl pl pt-br ru > tr zh-cn zh-tw > --------------------------------------------------------------- > /usr/local/bin/ruby ./tools/global-pgpkeys-creator.rb > HUGO_DISABLELANGUAGES=3D" bn-bd da de el es fr hu id it ja ko mn nl pl pt= -br > ru tr zh-cn zh-tw" /usr/local/bin/hugo --minify > Start building sites =E2=80=A6 > hugo v0.140.2+extended freebsd/amd64 BuildDate=3D2024-12-30T17:44:54Z+000= 0 > VendorInfo=3Dfreebsd > > > | EN > -------------------+------- > Pages | 1187 > Paginator pages | 0 > Non-page files | 0 > Static files | 856 > Processed images | 0 > Aliases | 1 > Cleaned | 0 > > Total in 200805 ms > > Where is that file generated? > > It=E2=80=99s not in a Makefile. Eventually rubygem-asciidoctor-diagram ca= lls it to > generate the diagram. Search for a file where asciidoctor-diagram is > defined as a plugin. I am travelling today and will share the details at > night. > The reason the image was not properly loaded is because the syntax was wrong in d7957d161fd when this change was introduced. I just fixed it in 6abcb860ac: -.Number of ports added between 1995 and 2022 [[fig-ports]] -image::portsstatus.svg +.Number of ports added between 1995 and 2022 +[[fig-ports]] +image::portsstatus.svg[Refer to paragraphs below for a screen-reader friendly version.] I don't have any gnuplot packages in my system: $ pkg info -x '.*plot.*' pkg: No package(s) matching .*plot.* and yet with that change "make run" now shows the picture in the documentation. The CI for the documentation tree needs fixing ( https://github.com/freebsd/freebsd-ci/pull/178) so the image might not show yet in docs.freebsd.org. The image in question is generated by https://people.freebsd.org/~bofh/dropzone/dev-model.sh, which reads at the beginning: # This script will inform the total number of committers per repo at # the end of the run. This information should be manually updated in # the file documentation/content/en/books/dev-model/_index.adoc According to git log, the generated image was last updated in 2022, and I can tell you I've built the whole tree a number of times since 2022 :-) Are you sure you uploaded it to the repo and hook it to the build? Cheers. > >> >> There is an SVG file that was supposed to be generated and it has >> nothing to do with the previous docproj. >> >> >> Kind regards, >> Moin >> > --0000000000001d980e062bead722 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr"><br></d= iv><br><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On = Thu, Jan 16, 2025 at 1:21=E2=80=AFPM Moin Rahman <<a href=3D"mailto:bofh= @freebsd.org" target=3D"_blank">bofh@freebsd.org</a>> wrote:<br></div><b= lockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-le= ft:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"auto"><div dir= =3D"ltr"><br></div><div dir=3D"ltr"><br><blockquote type=3D"cite">On 16 Jan= 2025, at 13:02, Fernando Apestegu=C3=ADa <<a href=3D"mailto:fernape@fre= ebsd.org" target=3D"_blank">fernape@freebsd.org</a>> wrote:<br><br></blo= ckquote></div><blockquote type=3D"cite"><div dir=3D"ltr">=EF=BB=BF<div dir= =3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote"><div dir= =3D"ltr" class=3D"gmail_attr">On Thu, Jan 16, 2025 at 8:56=E2=80=AFAM Moin = Rahman <<a href=3D"mailto:bofh@freebsd.org" target=3D"_blank">bofh@freeb= sd.org</a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"m= argin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left= :1ex"><br> <br> > On Jan 16, 2025, at 08:41, Fernando Apestegu=C3=ADa <fernape@FreeBS= D.org> wrote:<br> > <br> > The branch main has been updated by fernape:<br> > <br> > URL: <a href=3D"https://cgit.FreeBSD.org/ports/commit/?id=3Da8d3668988= 945bf23844a1733da214113e08d6ec" rel=3D"noreferrer" target=3D"_blank">https:= //cgit.FreeBSD.org/ports/commit/?id=3Da8d3668988945bf23844a1733da214113e08d= 6ec</a><br> > <br> > commit a8d3668988945bf23844a1733da214113e08d6ec<br> > Author:=C2=A0 =C2=A0 =C2=A0Fernando Apestegu=C3=ADa <fernape@FreeBS= D.org><br> > AuthorDate: 2025-01-14 17:26:26 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Fernando Apestegu=C3=ADa <fernape@FreeBS= D.org><br> > CommitDate: 2025-01-16 07:41:00 +0000<br> > <br> >=C2=A0 =C2=A0 textproc/docproj: remove gnuplot-lite dependency<br> > <br> >=C2=A0 =C2=A0 It was a leftover from the previous docproj.<br> > ---<br> > textproc/docproj/Makefile | 2 +-<br> > 1 file changed, 1 insertion(+), 1 deletion(-)<br> > <br> > diff --git a/textproc/docproj/Makefile b/textproc/docproj/Makefile<br> > index 6d362be1878a..a0b3c298573c 100644<br> > --- a/textproc/docproj/Makefile<br> > +++ b/textproc/docproj/Makefile<br> > @@ -1,5 +1,6 @@<br> > PORTNAME=3D docproj<br> > PORTVERSION=3D 4.1.9<br> > +PORTREVISION=3D 1<br> > CATEGORIES=3D textproc<br> > <br> > MAINTAINER=3D doceng@FreeBSD.org<br> > @@ -7,7 +8,6 @@ COMMENT=3D Meta-port for the FreeBSD Documentation Pro= ject<br> > WWW=3D <a href=3D"https://www.freebsd.org/docproj" rel=3D"noreferrer" = target=3D"_blank">https://www.freebsd.org/docproj</a><br>; > <br> > RUN_DEPENDS=3D asciidoctor:textproc/rubygem-asciidoctor \<br> > - gnuplot-lite>0:math/gnuplot-lite \<br> > gohugo>=3D0.133.1,1:www/gohugo \<br> > rougify:textproc/rubygem-rouge \<br> > rubygem-asciidoctor-diagram>0:textproc/rubygem-asciidoctor-diagram<= br> <br> I don't think this is correct. gnuplot was used to generate the graph<b= r> of number of ports here:<br> <br> <a href=3D"https://docs.freebsd.org/en/books/dev-model/#sub-project-ports" = rel=3D"noreferrer" target=3D"_blank">https://docs.freebsd.org/en/books/dev-= model/#sub-project-ports</a></blockquote><div><br></div><div>Interesting. I= don't see any invocations to gnuplot in any of the Makefiles.</div><di= v>Also, I don't have any gnuplot* packages in my system and I regularly= build the documentation just fine:</div><div><br></div><div>$ DOC_LANG=3De= n make<br>---------------------------------------------------------------<b= r>Building the documentation<br>included languages: en<br>excluded language= s: bn-bd da de el es fr hu id it ja ko mn nl pl pt-br ru tr zh-cn zh-tw<br>= ---------------------------------------------------------------<br>/usr/loc= al/bin/ruby ./tools/global-pgpkeys-creator.rb<br>HUGO_DISABLELANGUAGES=3D&q= uot; bn-bd da de el es fr hu id it ja ko mn nl pl pt-br ru tr zh-cn zh-tw&q= uot; /usr/local/bin/hugo --minify<br>Start building sites =E2=80=A6 <br>hug= o v0.140.2+extended freebsd/amd64 BuildDate=3D2024-12-30T17:44:54Z+0000 Ven= dorInfo=3Dfreebsd<br><br><br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0| =C2=A0EN =C2=A0 <br>-------------------+-------<b= r>=C2=A0 Pages =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 1187 =C2=A0<br>= =C2=A0 Paginator pages =C2=A0| =C2=A0 =C2=A00 =C2=A0<br>=C2=A0 Non-page fil= es =C2=A0 | =C2=A0 =C2=A00 =C2=A0<br>=C2=A0 Static files =C2=A0 =C2=A0 | = =C2=A0856 =C2=A0<br>=C2=A0 Processed images | =C2=A0 =C2=A00 =C2=A0<br>=C2= =A0 Aliases =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| =C2=A0 =C2=A01 =C2=A0<br>= =C2=A0 Cleaned =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| =C2=A0 =C2=A00 =C2=A0<br= ><br>Total in 200805 ms<br></div><div>=C2=A0</div><div>Where is that file g= enerated?</div></div></div></div></blockquote>It=E2=80=99s not in a Makefil= e. Eventually rubygem-asciidoctor-diagram calls it to generate the diagram.= Search for a file where asciidoctor-diagram is defined as a plugin. I am t= ravelling today and will share the details at night.</div></blockquote><div= ><br></div><br><div>The reason the image was not properly loaded is because= the syntax was wrong in d7957d161fd when this change was introduced.</div>= <div>I just fixed it in 6abcb860ac:</div><div><br></div><div>-.Number of po= rts added between 1995 and 2022 [[fig-ports]]<br>-image::portsstatus.svg<br= >+.Number of ports added between 1995 and 2022<br>+[[fig-ports]]<br>+image:= :portsstatus.svg[Refer to paragraphs below for a screen-reader friendly ver= sion.]<br></div><div><br></div><div>I don't have any gnuplot packages i= n my system:</div><div><br></div><div>$ pkg info -x '.*plot.*'<br>p= kg: No package(s) matching .*plot.*</div><div><br></div><div>and yet with t= hat change "make run" now shows the picture in the documentation.= </div><div>The CI for the documentation tree needs fixing (<a href=3D"https= ://github.com/freebsd/freebsd-ci/pull/178">https://github.com/freebsd/freeb= sd-ci/pull/178</a>) so the image might not show yet in <a href=3D"http://do= cs.freebsd.org">docs.freebsd.org</a>.<br></div><div><br></div><div><div>The= image in question is generated by <a href=3D"https://people.freebsd.org/~b= ofh/dropzone/dev-model.sh" target=3D"_blank">https://people.freebsd.org/~bo= fh/dropzone/dev-model.sh</a>, which reads at the beginning:</div><div></div= ><div><pre># This script will inform the total number of committers per rep= o at # the end of the run. This information should be manually updated in # the file documentation/content/en/books/dev-model/_index.adoc</pre></div>= <div>According to git log, the generated image was last updated in 2022, and I can tell you I've built the whole tree a number of times since 2022 :-)<br></di= v><div>Are you sure you uploaded it to the repo and hook it to the build?<b= r></div><div><br></div><div>Cheers.<br></div></div><div><br></div><blockquo= te class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px = solid rgb(204,204,204);padding-left:1ex"><div dir=3D"auto"><div><br><blockq= uote type=3D"cite"><div dir=3D"ltr"><div dir=3D"ltr"><div class=3D"gmail_qu= ote"><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;bo= rder-left:1px solid rgb(204,204,204);padding-left:1ex"><br> <br> There is an SVG file that was supposed to be generated and it has<br> nothing to do with the previous docproj.<br> <br> <br> Kind regards,<br> Moin<br> </blockquote></div></div> </div></blockquote></div></div></blockquote></div></div> </div> </div> --0000000000001d980e062bead722--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGwOe2Yy77o3kLtChNoAMszKzh2ecUDJHbs4k9mizhGOuv_a2A>