Date: Sat, 20 May 2006 15:34:26 +0200 From: Jean-Yves Lefort <jylefort@FreeBSD.org> To: Hiroki Sato <hrs@FreeBSD.org> Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/print/acroread7 Makefile pkg-deinstall pkg-plist.7.0.1 pkg-plist.7.0.5 ports/print/acroread7/files pkg-install.in Message-ID: <20060520153426.fa6d509b.jylefort@FreeBSD.org> In-Reply-To: <20060520.221521.122609088.hrs@allbsd.org> References: <20060519075844.e1ac8d3b.jylefort@FreeBSD.org> <20060519.154122.123975929.hrs@allbsd.org> <20060519090956.4f19dd41.jylefort@FreeBSD.org> <20060520.221521.122609088.hrs@allbsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--Signature=_Sat__20_May_2006_15_34_26_+0200_IcFRqp52rofaNmNB Content-Type: text/plain; charset=US-ASCII Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, 20 May 2006 22:15:21 +0900 (JST) Hiroki Sato <hrs@FreeBSD.org> wrote: > Jean-Yves Lefort <jylefort@FreeBSD.org> wrote > in <20060519090956.4f19dd41.jylefort@FreeBSD.org>: >=20 > jy> With that interpretation the port is also illegal since it's not only > jy> a "new installer", but also "modifies the Software" (through brandelf, > jy> for instance): > jy> > jy> "You may not otherwise alter or modify the Software or create a new > jy> installer for the Software." >=20 > I think neither the port is a new installer nor it modifies the Software, > and I think pkg-install script in your patch modifies the installed file. > This is my opinion as the port maintainer. >=20 > jy> Can I fix pkg-deinstall by checking if $PKG_PREFIX/Adobe/Acrobat7.0 is > jy> empty after deinstallation? >=20 > Why do you force me on your idea? I'm not forcing you, I just added a new point to the discussion (according to your interpretation, the port already violates the license) and asked you to reconsider in light of that addition. > I do not like using > pkg-install/pkg-deinstall to install the desktop entry > in the master port, and I think it should be done by using > symlink or so in another port (print/acroreadwrapper or new one), > or something like acroreadwrapper's --install-plugin option. > We also have to consider mixing of version 7.0.5 and 7.0.1. >=20 > I can understand installing the desktop entry and the icon file > is a good thing (IIRC, netchild@, mezz@, and I discussed this > topic before), but at least I have not approved your patch and > do not accept the installation/deinstallation scripts you committed. You have approved it: Approved by: maintainer timeout (17 days) Since you have changed your mind, I'll no longer annoy you with this. FYI, installing the .desktop file is desirable not only because of the menu item, but also because it associates the application/pdf MIME type with acroread, so that users don't have to do it manually. --=20 Jean-Yves Lefort jylefort@FreeBSD.org http://lefort.be.eu.org/ --Signature=_Sat__20_May_2006_15_34_26_+0200_IcFRqp52rofaNmNB Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.3 (FreeBSD) iD8DBQFEbxrnyzD7UaO4AGoRArVJAJsH9L07AVN8ytCio/76VCAyTkkbEQCdFYXD 5edIwNKqDJUXISdOuIUrOAk= =2Mgd -----END PGP SIGNATURE----- --Signature=_Sat__20_May_2006_15_34_26_+0200_IcFRqp52rofaNmNB--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060520153426.fa6d509b.jylefort>