Date: Mon, 31 Jul 2023 08:46:25 +0200 From: =?UTF-8?Q?Fernando_Apestegu=C3=ADa?= <fernape@freebsd.org> To: Alexey Dokuchaev <danfe@freebsd.org> Cc: "ports-committers@FreeBSD.org" <ports-committers@freebsd.org>, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 79e5fba255ef - main - cad/PrusaSlicer: update to 2.6.0 Message-ID: <CAGwOe2YvYmxnCMO6oJ2=3Vnc57hd8-RGO_sd1oGrR7XgfgXHJQ@mail.gmail.com> In-Reply-To: <ZMSg5A-a77EDobnZ@FreeBSD.org> References: <202307281248.36SCmERV027986@gitrepo.freebsd.org> <ZMO8UT68PlITqFuB@FreeBSD.org> <CAGwOe2YcXfogxwCODdHvOLPZExeXC_BN4knB3obFb_7EQUuA-g@mail.gmail.com> <ZMSg5A-a77EDobnZ@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000cc72740601c2ca1e Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Sat, Jul 29, 2023 at 7:17=E2=80=AFAM Alexey Dokuchaev <danfe@freebsd.org= > wrote: > On Fri, Jul 28, 2023 at 04:40:11PM +0200, Fernando Apestegu??a wrote: > > El vie, 28 jul 2023, 15:02, Alexey Dokuchaev escribi??: > > > On Fri, Jul 28, 2023 at 12:48:14PM +0000, Fernando Apestegu??a wrote: > > > > commit 79e5fba255ef2620dd872e21bdd7dc667fea3c19 > > > > > > > > cad/PrusaSlicer: update to 2.6.0 > > > > > > > > PR: 272730 > > > > Reported by: michal.zielonka.8001@gmail.com > > > > Approved by: teodorsigaev@gmail.com (maintainer) > > > > --- > > > > cad/PrusaSlicer/Makefile | 14 +- > > > > cad/PrusaSlicer/distinfo | 6 +- > > > > cad/PrusaSlicer/files/patch-CMakeLists.txt | 19 +- > > > > cad/PrusaSlicer/files/patch-src_CMakeLists.txt | 15 +- > > > > cad/PrusaSlicer/files/patch-src_avrdude_arduino.c | 10 + > > > > .../files/patch-src_libnanosvg_nanosvg.h | 3109 > > > ++++++++++++++++++++ > > > > .../files/patch-src_libnanosvg_nanosvgrast.h | 1485 ++++++++= ++ > > > > > > Who reviewed this? First, we don't create files via patches, and > > > > Maintainer did. It's in the commit log and in the PR. > > Which means they don't have a clue, and you as committer should have > requested this whole mess reworked, not checked in as is. These are > the basics your mentor should have explained, did he not? > You can contact them and ask them. I assume you know how to get that information. About how to contribute to FreeBSD, well, I **don't do** patch review/build testing in all supported platforms I can get my hands on/run testing in clean environments/modernization of old ports/train submitters/etc. But I *keep my fingers crossed when I do git push*. That has to count for something right? Have a nice day. > > ./danfe > --000000000000cc72740601c2ca1e Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <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 Sat, Jul 29, 2023 at 7:17=E2=80=AF= AM Alexey Dokuchaev <<a href=3D"mailto:danfe@freebsd.org">danfe@freebsd.= org</a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"marg= in:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1e= x">On Fri, Jul 28, 2023 at 04:40:11PM +0200, Fernando Apestegu??a wrote:<br= > > El vie, 28 jul 2023, 15:02, Alexey Dokuchaev escribi??:<br> > > On Fri, Jul 28, 2023 at 12:48:14PM +0000, Fernando Apestegu??a wr= ote:<br> > > > commit 79e5fba255ef2620dd872e21bdd7dc667fea3c19<br> > > ><br> > > >=C2=A0 =C2=A0cad/PrusaSlicer: update to 2.6.0<br> > > ><br> > > >=C2=A0 =C2=A0PR:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0272730<br> > > >=C2=A0 =C2=A0Reported by:=C2=A0 =C2=A0 <a href=3D"mailto:mich= al.zielonka.8001@gmail.com" target=3D"_blank">michal.zielonka.8001@gmail.co= m</a><br> > > >=C2=A0 =C2=A0Approved by:=C2=A0 =C2=A0 <a href=3D"mailto:teod= orsigaev@gmail.com" target=3D"_blank">teodorsigaev@gmail.com</a> (maintaine= r)<br> > > > ---<br> > > >=C2=A0 cad/PrusaSlicer/Makefile=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2= =A0 =C2=A014 +-<br> > > >=C2=A0 cad/PrusaSlicer/distinfo=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2= =A0 =C2=A0 6 +-<br> > > >=C2=A0 cad/PrusaSlicer/files/patch-CMakeLists.txt=C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A019 +-<br> > > >=C2=A0 cad/PrusaSlicer/files/patch-src_CMakeLists.txt=C2=A0 = =C2=A0 =C2=A0|=C2=A0 =C2=A015 +-<br> > > >=C2=A0 cad/PrusaSlicer/files/patch-src_avrdude_arduino.c=C2= =A0 |=C2=A0 =C2=A010 +<br> > > >=C2=A0 .../files/patch-src_libnanosvg_nanosvg.h=C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3109<br> > > ++++++++++++++++++++<br> > > >=C2=A0 .../files/patch-src_libnanosvg_nanosvgrast.h=C2=A0 =C2= =A0 =C2=A0 =C2=A0| 1485 ++++++++++<br> > ><br> > > Who reviewed this?=C2=A0 First, we don't create files via pat= ches, and<br> > <br> > Maintainer did. It's in the commit log and in the PR.<br> <br> Which means they don't have a clue, and you as committer should have<br= > requested this whole mess reworked, not checked in as is.=C2=A0 These are <= br></blockquote><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0= px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> the basics your mentor should have explained, did he not?<br></blockquote><= /div><div class=3D"gmail_quote"><br></div><div class=3D"gmail_quote">You ca= n contact them and ask them. I assume you know how to get that information.= </div><div class=3D"gmail_quote"><br></div><div class=3D"gmail_quote">About= how to contribute to FreeBSD, well, I **don't do** patch review/build = testing in all supported platforms I can get my hands on/run testing in cle= an environments/modernization of old ports/train submitters/etc.</div><div = class=3D"gmail_quote">But I *keep my fingers crossed when I do git push*. T= hat has to count for something right?</div><div class=3D"gmail_quote"><br><= /div><div class=3D"gmail_quote">Have a nice day.<br></div><div class=3D"gma= il_quote"><div>=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"margi= n:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex= "> <br> ./danfe<br> </blockquote></div></div> --000000000000cc72740601c2ca1e--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGwOe2YvYmxnCMO6oJ2=3Vnc57hd8-RGO_sd1oGrR7XgfgXHJQ>