From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id BCC321396D0 for ; Mon, 18 Sep 2017 12:02:13 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 91883E0EAA; Mon, 18 Sep 2017 12:02:04 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 3A381E0EA5 for ; Mon, 18 Sep 2017 12:02:04 +0000 (UTC) Received: from rubberducky (c-174-56-100-228.hsd1.nm.comcast.net [174.56.100.228]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: nicolasbock) by smtp.gentoo.org (Postfix) with ESMTPSA id 441CB33BE69 for ; Mon, 18 Sep 2017 12:02:03 +0000 (UTC) Date: Mon, 18 Sep 2017 06:02:00 -0600 From: Nicolas Bock To: gentoo-dev@lists.gentoo.org Subject: Re: [gentoo-dev] New package neomutt Message-ID: <20170918115843.lugsc7dtrjrn72by@rubberducky> References: <20170731071119.jccco5q4kd3fs4xs@rubberducky.suse.de> <20170810045857.e6qrvnimteopxgev@rubberducky.suse.de> <1502350830.1554.1.camel@gentoo.org> Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="wug45uvlh2644j5l" Content-Disposition: inline In-Reply-To: <1502350830.1554.1.camel@gentoo.org> User-Agent: NeoMutt/20170714 (1.8.3) X-Archives-Salt: f2911a20-695b-47cc-9c51-59c9b3a1098f X-Archives-Hash: 43f9b72678ed2b36ad5ab56df5e4f12e --wug45uvlh2644j5l Content-Type: text/plain; charset=utf-8; format=flowed Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Aug 10, 2017 at 09:40:30AM +0200, Micha=C5=82 G=C3=B3rny wrote: >On czw, 2017-08-10 at 06:58 +0200, Nicolas Bock wrote: >> On Mon, Jul 31, 2017 at 09:11:19AM +0200, Nicolas Bock wrote: >>> Hi, >>> >>> I would like to add neomutt to the tree. This new package is meant as >>> an alternative and not a replacement of the existing mutt package. >> >> Thanks for all of the great suggestions and feedback! >> >> This is round two. I have update the ebuild with all your >> suggestions. I have also added support for eselecting between mutt >> and neomutt. Before the eselect ebuild can land though, we need to >> rename the mutt binary so that the managed link can be called >> mutt. > >What for? How many people are exactly in the dire need of having both >installed simultaneously and switching between them? If you really can't >learn to type the new command, add IUSE=3Dsymlink blocking original mutt >and be done with it. Don't add more unowned files to /usr by another >poorly written eselect module. I changed the ebuild such that it can be installed together with=20 mutt without having to use symlinks or eselet. >> # Copyright 1999-2017 Gentoo Foundation >> # Distributed under the terms of the GNU General Public License v2 >> >> EAPI=3D6 >> >> inherit autotools eutils flag-o-matic >> >> if [[ ${PV} =3D=3D 99999999 ]] ; then >> # live ebuild >> inherit git-r3 >> EGIT_REPO_URI=3D"https://github.com/neomutt/neomutt.git" >> EGIT_CHECKOUT_DIR=3D"${WORKDIR}/neomutt-${P}" >> KEYWORDS=3D"" > >This is going to confuse the hell out of ekeyword. I split the ebuild into two so that this conditional is unnecessary. >> else >> SRC_URI=3D"https://github.com/${PN}/${PN}/archive/${P}.tar.gz" >> KEYWORDS=3D"~amd64 ~x86" >> fi >> >> DESCRIPTION=3D"Teaching an Old Dog New Tricks" > >This doesn't tell anybody who doesn't know mutt what this is. Fixed, I reverted that change. >> HOMEPAGE=3D"https://www.neomutt.org/" >> >> LICENSE=3D"GPL-2" >> SLOT=3D"0" >> IUSE=3D"berkdb crypt debug doc gdbm gnutls gpg idn kerberos libressl mbox >> nls notmuch qdbm sasl selinux slang smime ssl tokyocabinet kyotocabinet >> lmdb" > >Sort lexically. Fixed. >> >> CDEPEND=3D" >> app-eselect/eselect-mutt >> app-misc/mime-types >> nls? ( virtual/libintl ) >> tokyocabinet? ( dev-db/tokyocabinet ) >> qdbm? ( dev-db/qdbm ) >> gdbm? ( sys-libs/gdbm ) >> berkdb? ( >=3Dsys-libs/db-4:=3D ) >> kyotocabinet? ( dev-db/kyotocabinet ) >> lmdb? ( dev-db/lmdb ) >> gnutls? ( >=3Dnet-libs/gnutls-1.0.17 ) >> !gnutls? ( >> ssl? ( >> !libressl? ( >=3Ddev-libs/openssl-0.9.6:0 ) >> libressl? ( dev-libs/libressl ) >> ) >> ) > >This is not a correct use of 'ssl' flag: > >=C2=A0global:ssl: Add support for Secure Socket Layer connections > >It's supposed to go top-level, above any implementation flags. > >> sasl? ( >=3Ddev-libs/cyrus-sasl-2 ) >> kerberos? ( virtual/krb5 ) >> idn? ( net-dns/libidn ) >> gpg? ( >=3Dapp-crypt/gpgme-0.9.0 )=09 >> smime? ( >> !libressl? ( >=3Ddev-libs/openssl-0.9.6:0 ) >> libressl? ( dev-libs/libressl ) > >What is the point of preferring gnutls when USE=3Dsmime pulls openssl >anyway? > >> ) >> notmuch? ( net-mail/notmuch ) >> slang? ( sys-libs/slang ) >> !slang? ( >=3Dsys-libs/ncurses-5.2:0 ) > >Why not =3D slotop? ncurses definitely changed ABI in the past. It's >something you are supposed to use when needed, not when repoman >complains about it and you didn't accidentally workaround the check. The current version of ncurses works, so I removed the version on it. I also sorted the dependencies. >Sorting this would also help reviews. > >> " >> DEPEND=3D"${CDEPEND} >> net-mail/mailbase >> doc? ( >> dev-libs/libxml2 >> dev-libs/libxslt >> app-text/docbook-xsl-stylesheets >> || ( www-client/lynx www-client/w3m www-client/elinks ) >> )" >> RDEPEND=3D"${CDEPEND} >> selinux? ( sec-policy/selinux-mutt ) >> " >> >> S=3D"${WORKDIR}/${PN}-${P}" >> >> src_prepare() { >> eapply "${FILESDIR}/0001-Rename-mutt-to-neomutt.patch" >> eapply_user >> AT_M4DIR=3D"m4" eautoreconf >> } >> >> src_configure() { >> local myconf=3D( >> "$(use_enable crypt pgp)" >> "$(use_enable debug)" >> "$(use_enable doc)" >> "$(use_enable gpg gpgme)" >> "$(use_enable nls)" >> "$(use_enable smime)" >> "$(use_enable notmuch)" >> "$(use_with idn)" >> "$(use_with kerberos gss)" >> "$(use_with sasl)" >> "$(use_with tokyocabinet)" >> "$(use_with kyotocabinet)" >> "$(use_with qdbm)" >> "$(use_with gdbm)" >> "$(use_with berkdb bdb)" >> "$(use_with lmdb)" >> "--with-$(use slang && echo slang || echo curses)=3D${EPREFIX}/usr" > >usex Fixed >> "--sysconfdir=3D${EPREFIX}/etc/${PN}" > >I'd really prefer if you didn't abuse PN to construct paths, and make me >wonder if upstream really wants 'neomutt' or 'mutt' here. Up until now upstream was using `mutt` but there is work in progress to rename it to `neomutt`. I think that there is a legitimate use case for installing `mutt` and `neomutt` at the same time. >> "--with-docdir=3D${EPREFIX}/usr/share/doc/${PN}-${PVR}" > >PF? Fixed. >> ) >> >> if [[ ${CHOST} =3D=3D *-solaris* ]] ; then >> # arrows in index view do not show when using wchar_t >> myconf+=3D( "--without-wc-funcs" ) >> fi > >Are you sure that this still applies? No, and as I don't have access to such a machine I can't test it. >> >> # there's no need for gnutls, ssl or sasl without socket support > >What is this comment supposed to mean? Looks like copy-paste without >even reading it. This was a copy-and-paste leftover. I have removed it. I don't know what it's supposed to mean either. >> if use gnutls; then >> myconf+=3D( "--with-gnutls" ) >> elif use ssl; then >> myconf+=3D( "--with-ssl" ) >> fi >> >> if use mbox; then >> myconf+=3D( "--with-mailpath=3D${EPREFIX}/var/spool/mail" ) >> else >> myconf+=3D( "--with-homespool=3DMaildir" ) >> fi > >Would configuring both paths do any harm? Maybe it'd make easier for >user to switch without having to rebuild it. Good point. I have removed the conditional. I also removed the default configuration files. >> >> econf "${myconf[@]}" >> } >> >> src_install() { >> emake DESTDIR=3D"${D}" install >> if use mbox; then >> insinto /etc/${PN} > >You can move the insinto above the 'if'. I have removed this part. >> newins "${FILESDIR}"/Muttrc.mbox Muttrc >> else >> insinto /etc/${PN} >> doins "${FILESDIR}"/Muttrc >> fi >> >> # A newer file is provided by app-misc/mime-types. So we link it. >> rm "${ED}"/etc/${PN}/mime.types || die >> dosym /etc/mime.types /etc/${PN}/mime.types > >Don't use absolute symlinks. Fixed. >> >> # A man-page is always handy, so fake one >> if use !doc; then >> emake -C doc DESTDIR=3D"${D}" muttrc.man > >Is DESTDIR really necessary here? No, fixed. >> # make the fake slightly better, bug #413405 >> sed -e 's#@docdir@/manual.txt#http://www.mutt.org/doc/manual/#' \ >> -e 's#in @docdir@,#at http://www.mutt.org/,#' \ > >You sure you want to link to the original mutt.org? True, I changed the link to point to neomutt. >> -e "s#@sysconfdir@#${EPREFIX}/etc/${PN}#" \ >> -e "s#@bindir@#${EPREFIX}/usr/bin#" \ >> doc/mutt.man > neomutt.1 || die >> cp doc/muttrc.man neomuttrc.5 || die >> doman neomutt.1 neomuttrc.5 >> else >> # nuke manpages that should be provided by an MTA, bug #177605 >> rm "${ED}"/usr/share/man/man5/{mbox,mmdf}.5 \ >> || ewarn "failed to remove files, please file a bug" > >die. Fixed. >> fi >> >> dodoc COPYRIGHT ChangeLog* OPS* README* >> } --=20 Nicolas Bock --wug45uvlh2644j5l Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQKTBAABCgB9FiEEU2lNM4ZQY0eES+u+HcDb5tcV6qcFAlm/tbhfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDUz Njk0RDMzODY1MDYzNDc4NDRCRUJCRTFEQzBEQkU2RDcxNUVBQTcACgkQHcDb5tcV 6qeu6RAAo62Fxz5PT6ZIg9O/v9vsU6xlm+Hvn2FH8oRPJTIZ4596FVrKlUluMj/z rzVyfc1el2JHoSUo13OSwtwh5qu9u2ZMrQ1M6+OoVWkF02YUT7JGf4nKH6sPI802 Elfj5M2oZEH1clwEgWbNhhz6ylhSgrlDobS/+wuWs8L4sTMcpZ+t2xd3ONkDJk5V FKoEmkEJQ6iIYzgpLcSWxO05anhOd13eiFQ9hW/SiYysWCnDGYdOBW350Walb/6O 5ALCck/Ip/agzyPDagXYzLRNPtsOXr19DplXE87vlGxqfzt4gz/9fN+7bX5ajwnC yr+TZiDOVjluWH41gHpUdMc8PsepW25RSzR16KDAQMqzaLbUTdG3NH19CUF31fZY grdOniiWC6oljswYWoAvnP4UciQTygmW3TeL4HQ+zIhPPtH9qBmFOR2QcWUX+HoR YK8gUspGoN7oYhTaAnvSPAgqStofUf/sv7IMprIX/vbQRTZGEKl6s5HfUhlJNQA7 CpdlgcU9nmGlp532BfnW8wVzm9a1PdZyMA+D612YSUN0/SESTCvJhGZzmAzWA+ZB exA28owOXnc+EXV1Zc7KNZJtWCr++y/vGr0tWImHFs7f6qVp0PkYQKO+7d290J/B GzDJjP9XH+vJBb/1RATIWGqIxKklmPvyrD0LyQQ674Ev0afMpJg= =E9jF -----END PGP SIGNATURE----- --wug45uvlh2644j5l--