Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 28 Jul 2023 16:40:11 +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:  <CAGwOe2YcXfogxwCODdHvOLPZExeXC_BN4knB3obFb_7EQUuA-g@mail.gmail.com>
In-Reply-To: <ZMO8UT68PlITqFuB@FreeBSD.org>
References:  <202307281248.36SCmERV027986@gitrepo.freebsd.org> <ZMO8UT68PlITqFuB@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000009730fe06018d0f18
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

El vie, 28 jul 2023, 15:02, Alexey Dokuchaev <danfe@freebsd.org> escribi=C3=
=B3:

> 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.


second, why not properly depend on `graphics/nanosvg' port?
>
> > diff --git a/cad/PrusaSlicer/files/patch-src_libnanosvg_nanosvg.h
> b/cad/PrusaSlicer/files/patch-src_libnanosvg_nanosvg.h
> > new file mode 100644
> > index 000000000000..08bf0beb1ef1
> > --- /dev/null
> > +++ b/cad/PrusaSlicer/files/patch-src_libnanosvg_nanosvg.h
> > @@ -0,0 +1,3109 @@
> > +--- src/libnanosvg/nanosvg.h.orig    1970-01-01 01:00:00.000000000 +01=
00
> > ++++ src/libnanosvg/nanosvg.h 2022-12-22 00:42:08.000000000 +0100
> > +@@ -0,0 +1,3106 @@
> > ++/*
> > ++ * Copyright (c) 2013-14 Mikko Mononen memon@inside.org
> > ...
>
> ./danfe
>

--0000000000009730fe06018d0f18
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" =
class=3D"gmail_attr">El vie, 28 jul 2023, 15:02, Alexey Dokuchaev &lt;<a hr=
ef=3D"mailto:danfe@freebsd.org">danfe@freebsd.org</a>&gt; escribi=C3=B3:<br=
></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-=
left:1px #ccc solid;padding-left:1ex">On Fri, Jul 28, 2023 at 12:48:14PM +0=
000, Fernando Apestegu??a wrote:<br>
&gt; commit 79e5fba255ef2620dd872e21bdd7dc667fea3c19<br>
&gt; <br>
&gt;=C2=A0 =C2=A0cad/PrusaSlicer: update to 2.6.0<br>
&gt; <br>
&gt;=C2=A0 =C2=A0PR:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0272730<=
br>
&gt;=C2=A0 =C2=A0Reported by:=C2=A0 =C2=A0 <a href=3D"mailto:michal.zielonk=
a.8001@gmail.com" target=3D"_blank" rel=3D"noreferrer">michal.zielonka.8001=
@gmail.com</a><br>
&gt;=C2=A0 =C2=A0Approved by:=C2=A0 =C2=A0 <a href=3D"mailto:teodorsigaev@g=
mail.com" target=3D"_blank" rel=3D"noreferrer">teodorsigaev@gmail.com</a> (=
maintainer)<br>
&gt; ---<br>
&gt;=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>
&gt;=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>
&gt;=C2=A0 cad/PrusaSlicer/files/patch-CMakeLists.txt=C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0|=C2=A0 =C2=A019 +-<br>
&gt;=C2=A0 cad/PrusaSlicer/files/patch-src_CMakeLists.txt=C2=A0 =C2=A0 =C2=
=A0|=C2=A0 =C2=A015 +-<br>
&gt;=C2=A0 cad/PrusaSlicer/files/patch-src_avrdude_arduino.c=C2=A0 |=C2=A0 =
=C2=A010 +<br>
&gt;=C2=A0 .../files/patch-src_libnanosvg_nanosvg.h=C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0| 3109 ++++++++++++++++++++<br>
&gt;=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&#39;t create files via patches, and<=
br></blockquote></div></div><div dir=3D"auto"><br></div><div dir=3D"auto">M=
aintainer did. It&#39;s in the commit log and in the PR.</div><div dir=3D"a=
uto"><br></div><div dir=3D"auto"><br></div><div dir=3D"auto"><div class=3D"=
gmail_quote"><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;b=
order-left:1px #ccc solid;padding-left:1ex">
second, why not properly depend on `graphics/nanosvg&#39; port?<br>
<br>
&gt; diff --git a/cad/PrusaSlicer/files/patch-src_libnanosvg_nanosvg.h b/ca=
d/PrusaSlicer/files/patch-src_libnanosvg_nanosvg.h<br>
&gt; new file mode 100644<br>
&gt; index 000000000000..08bf0beb1ef1<br>
&gt; --- /dev/null<br>
&gt; +++ b/cad/PrusaSlicer/files/patch-src_libnanosvg_nanosvg.h<br>
&gt; @@ -0,0 +1,3109 @@<br>
&gt; +--- src/libnanosvg/nanosvg.h.orig=C2=A0 =C2=A0 1970-01-01 01:00:00.00=
0000000 +0100<br>
&gt; ++++ src/libnanosvg/nanosvg.h 2022-12-22 00:42:08.000000000 +0100<br>
&gt; +@@ -0,0 +1,3106 @@<br>
&gt; ++/*<br>
&gt; ++ * Copyright (c) 2013-14 Mikko Mononen <a href=3D"mailto:memon@insid=
e.org" target=3D"_blank" rel=3D"noreferrer">memon@inside.org</a><br>
&gt; ...<br>
<br>
./danfe<br>
</blockquote></div></div></div>

--0000000000009730fe06018d0f18--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGwOe2YcXfogxwCODdHvOLPZExeXC_BN4knB3obFb_7EQUuA-g>