From: Jon Turney <jon.turney@dronecode.org.uk>
To: as@php.net, "cygwin-apps@cygwin.com" <cygwin-apps@cygwin.com>
Subject: Re: [ITP] mle
Date: Tue, 7 Jun 2022 19:18:42 +0100 [thread overview]
Message-ID: <657aca46-294f-d12f-db27-c4fb4cde2be8@dronecode.org.uk> (raw)
In-Reply-To: <CAB-ec3RFmqx4V6r1e1hNukCLEtRAJNBwf+kSK5c4dfNto6EG_g@mail.gmail.com>
On 06/06/2022 15:33, as-GOAhiB59KME@public.gmane.org wrote:
> Hello all,
>
> I'd like to submit a cygport for mle[0], a text editor. It's available
> on various distros/OSs[1] including Debian, Fedora, FreeBSD, and
> others. It's licensed under ASL 2.0.
Thanks.
> A first attempt at a cygport is here[2]. The editor depends on a
> header-only library called uthash which I also attempted to package
> here[3]. The other deps, libpcre2-devel and liblua-devel, are already
> packaged. Is there a way to express a version constraint on the Lua
> dep? It needs 5.4.
Unfortunately, there is not currently.
A few comments on the cygport files below.
> [0] https://github.com/adsr/mle
> [1] https://repology.org/project/mle/versions
> [2] https://gist.github.com/adsr/4ecb4113f2d9c0e4ee389a800c0fd15b
> NAME="mle"
> VERSION=1.5.0
> HASH="bc05ec0eee4143d824010c6688fce526550ed508"
> RELEASE=1
> CATEGORY="Editors"
> SUMMARY="Flexible terminal-based text editor"
> DESCRIPTION="mle is a small, flexible, terminal-based text editor written in C.
> Notable features include: full Unicode support, syntax highlighting, scriptable
> rc file, macros, search and replace (PCRE), window splitting, multiple cursors,
> and integration with various shell commands."
> HOMEPAGE="https://github.com/adsr/mle"
> SRC_URI="https://github.com/adsr/mle/archive/${HASH}.tar.gz"
Is there some reason why this can't be
"https://github.com/adsr/mle/archive/refs/tags/v1.5.0.tar.gz" ?
> SRC_DIR="mle-${HASH}"
> BUILD_REQUIRES="libpcre2-devel uthash-devel liblua-devel"
> USE_DESTDIR=1
I don't think setting USE_DESTDIR=1 is needed as this is the default.
>
> src_compile() {
> cygmake -j1
If this is because the build system can't handle a parallel make, a
comment to that effect here would be good.
> }
>
> src_install() {
> cd ${S}
> cyginstall prefix=/usr
> }
>
> mle_CONTENTS="usr/bin/mle.exe usr/share/doc usr/lib/debug usr/src/debug"
This shouldn't list usr/{lib,src}/debug (as those belong in a debuginfo
package), and I don't think you need this line at all, as the default
will do the right thing?
> [3] https://gist.github.com/adsr/e5e6beb373a427dca9e3093cc31b4cea
> NAME="uthash-devel"
Don't do this, because this becomes the name of the source package as well.
Use NAME="uthash" and then PKG_NAMES="uthash-devel"
> VERSION=2.3.0
> RELEASE=1
> CATEGORY="Libs"
> SUMMARY="C macros for hash tables and more"
> DESCRIPTION="uthash is a collection of header-only data-structure libraries for
> hash maps (uthash), linked lists (utlist), dynamic arrays (utarray), dynamic
> ring buffers (utringbuffer), intrusive stacks (utstack), and dynamic strings
> (utstring)."
> HOMEPAGE="https://troydhanson.github.io/uthash/"
> SRC_URI="https://github.com/troydhanson/uthash/archive/refs/tags/v${VERSION}.tar.gz"
> ARCH="noarch"
>
> src_compile() {
> true
> }
>
> src_install() {
> cd ${S}/src
> doinclude uthash.h utlist.h utarray.h utringbuffer.h utstack.h utstring.h
> }
next prev parent reply other threads:[~2022-06-07 18:18 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-06 14:33 as
2022-06-07 18:18 ` Jon Turney [this message]
2022-06-08 1:58 ` as
2022-06-09 16:58 ` Jon Turney
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=657aca46-294f-d12f-db27-c4fb4cde2be8@dronecode.org.uk \
--to=jon.turney@dronecode.org.uk \
--cc=as@php.net \
--cc=cygwin-apps@cygwin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).