On 26 Jan 2018, at  4:35  -0500, Eli Schwartz via aur-general wrote:
> On 01/26/2018 04:23 PM, Ivy Foster wrote:
> > I'm writing to apply to be a TU, and Alad Wenter has kindly agreed to
> > be my sponsor.

> It is great to see you take the plunge, I wish you the best of luck!

Thanks!

> > Arch has always been a rewarding community to contribute to, and I
> > figure that maintaining some packages and generally helping out could
> > be a good way to contribute a bit more.
> > 
> > If accepted to be a TU, my plan of action is as follows:
> > 1. Go mad with power^U
> > 1. Bring a handful of packages into [community] (see below)
> > 2. Help out with rebuilds and package updates where that does not
> >     involve stepping on toes
> > 3. Continue to submit occasional patches to Arch projects
> > 4. Help with to-do lists. [...]

> Sounds like a (wo)man after my own heart!

Woman, and glad to hear it!

> This reminds me I still have so much to do... like all that https/gpg stuff.

There's always more to do, I guess.

> > Thanks for your consideration, and I'm of course happy to answer
> > questions and address critiques.

> But overall, quite good!

Thanks!

> > - ledger
> >     This program is super useful, and I doubt I'm the only one who
> >     dreads every boost update because this takes so long to build!

> Lukas has beaten you to it: https://packages.archlinux.org/ledger

I replied to this elsewhere already, but that's great news (-: . In
related news, I've [poked upstream][1] to see about a new release, since
there's been a year's worth of bugfixes! They're into it.

# Critiques & Responses

> We discussed this on IRC already, I'll have to check and see how you've
> adapted to my suggestions.

I've addressed most of them; see responses inline. Of course,
onlookers should judge each fix to make sure it's not a "fix" instead.

## cgo-git

> 2018-01-25 07:07:51 PM        guys    I noticed something immediately, 
> cgo-git has
> a custom:cgo-git license, but it is really an ISC license.
> 2018-01-25 07:08:15 PM        guys    And it installs the whole source code in
> /usr/share/licenses/ instead of using sed to extract it or something. :p
> 2018-01-25 07:09:25 PM        guys    I'd just extract the first few lines 
> using
> sed, until I hit the first  */ and call it a day

I've changed the license to ISC and used sed to extract the license
from the source. I've also [submitted a patch upstream][2] creating a
separate LICENSE file.

> 2018-01-25 07:11:25 PM        guys    Also, the upstream Makefile is terrible 
> and
> should use CFLAGS properly :p
> 2018-01-25 07:12:27 PM        guys    I want pull requests to fix this :p

[Pull request submitted][3].

> 2018-01-25 07:14:28 PM        guys    fist, should be upgraded to use HTTPS 
> since
> their website upgrades you anyway

[Done][4].

## frotz-dumb-git / frotz-ncurses-git

> 2018-01-25 07:27:55 PM        guys    frotz-git conflicts and *replaces* 
> frotz,
> which is wrong, it should provide it instead
> 2018-01-25 07:28:23 PM        guys    replaces means that if you pacman -Syu 
> and
> find it in a repo, it gets synced as a replacement for what you
> currently have...

> 2018-01-25 07:29:11 PM        guys    I can hardly read the sed line you use 
> in
> pkgver()
> 2018-01-25 07:29:22 PM        guys    sed 's,-\(.*\)-,.r\1.,'
> 2018-01-25 07:29:30 PM        guys    wrong place to use , as separators!
> 2018-01-25 07:33:03 PM        guys    But anyway, to modify 2.44-196-gf3ceac9
> could just use the standard sed line from the wiki page
> 2018-01-25 07:34:05 PM        escondida       that one *I* can hardly read (-:

> 2018-01-25 07:35:48 PM        guys    Use of sed to modify more than three 
> things
> in prepare should be strictly prohibited; use a patch file

[All fixed][5]. I still refuse to use the sed line from the wiki page,
because I'm a big weirdo.

## libbulletml & rrootage

> 2018-01-25 07:39:48 PM        guys    > # upstream does not provide checksums,
> though Debian does for their patches
> 2018-01-25 07:40:04 PM        guys    This is not a reason to disable checks 
> for
> download errors.

I've [added checksums][6], along with a note not to place too much
trust in them since they're mine and not the developer's.

> 2018-01-25 07:41:09 PM        guys    Why does libbulletml.so need to modify
> CFLAGS CXXFLAGS :(
> 2018-01-25 07:41:21 PM        guys    And why does it overwrite LDFLAGS, 
> instead?
> 2018-01-25 07:41:41 PM        guys    Does it derp on the LDFLAGS from 
> makepkg.conf?
> 2018-01-25 07:42:17 PM        guys    Why does it create libbulletml.a 
> anyway, if
> makepkg automatically strips staticlibs?

This library's build process is simply bizarre. I've left the build as
is, since that seems to be what it takes to get it to, well, build.

> 2018-01-25 07:51:23 PM        guys    rrootage: the pkgdesc is 
> self-referential,
> remove the first two words
> 2018-01-25 07:52:12 PM        guys    And it downloads from
> http://downloads.sourceforge.net, but that can be upgraded to https://
> even if the website cannot
> 2018-01-25 07:53:05 PM        guys    Is http://ftp.de.debian.org/ really the 
> best
> download url? I would assume they have something that redirects to the
> right country mirror

I found a better Debian download url and upgraded the Sourceforge url
to https; [fixes here][7].

I've also sent an email to the mysterious Evil Mr. Henry, the porter
(portreeve?) who brought it to Linux, to see whether he'd be
interested in accepting the Debian patches officially, since they're
required to build these days.

## lua-cosmo-git

> 2018-01-25 07:49:02 PM        guys    lua-cosmo-git: is built, in package().
> 2018-01-25 07:49:27 PM        guys    If it cannot be built separately from 
> the
> install command, build it in build() and cp -a it in package()
> 2018-01-25 07:50:12 PM        escondida       Oh, right, I meant to drop that 
> one.
> Instead, I'll update it and then drop it.

To be honest, I ended up simply dropping this package. As far as I can
tell, luarocks really seems to expect you to build and intall in one
step. I can pick it back up again if anybody has particularly strong
feelings on this point.

## netsurf et al.

> 2018-01-25 07:43:58 PM        guys    libcss-git should probably use 
> git+https://
> for TLS security purposes
> 2018-01-25 07:45:31 PM        guys    Same with the other netsurf related 
> packages

Sadly, I got SSL errors when I tried to pull netsurf libs from https.
They seem to be available only over http for now. I didn't try *all*
of them, granted, but I figure what's true for the css is true for the
cssander.

I need to update these in general, to tell the truth. Part of me wants
to do them all as a massive split PKGBUILD, because the dependency
order is fairly intricate (I have them in directories numbered in
stages on my machine, but that doesn't translate well to the AUR).

## sndio-git

> 2018-01-25 07:56:23 PM        guys    Is
> sndio-git/0001-put-cookie-somewhere-better.patch upstreamable?
> 2018-01-25 07:56:32 PM        guys    Or have you tried and they rejected it

I haven't written the better version to submit yet; that'll probably
be sometime in the next few days.

## translate-shell

> 2018-01-25 07:56:59 PM        guys    http://www.soimort.org/translate-shell/ 
> is
> available over HTTPS
> 2018-01-25 07:57:00 PM        phrik   Title: Translate Shell (at 
> www.soimort.org)
> 2018-01-25 07:57:09 PM        guys    As is the github repo via git+https:// 
> ;)

> 2018-01-25 07:57:29 PM        guys    pkgver=v0.9.0.4.7.g1a4d83e # should 
> have the
> leading v stripped

> https://github.com/soimort/translate-shell/blob/develop/Makefile
> 2018-01-25 07:59:10 PM        phrik   Title: translate-shell/Makefile at 
> develop
> · soimort/translate-shell · GitHub (at github.com)
> 2018-01-25 07:59:22 PM        guys    latest commit to that file says 
> "Implement
> DESTDIR support"
> 2018-01-25 07:59:39 PM        guys    Complete with mkdir -p ;)

> 2018-01-25 08:00:44 PM        guys    And 
> /usr/share/licenses/${_gitname}/LICENSE
> is technically quite wrong, that is not the package the license is for
> at all

> 2018-01-25 08:00:57 PM        guys    besides which it does not 
> conflict/provide
> the non-git version

[All fixed][8].

# Concluding thoughts

Who's next? Bring it on!

iff

# Links

[1]: https://github.com/ledger/ledger/issues/523
[2]: https://github.com/kieselsteini/cgo/pull/10
[3]: https://github.com/kieselsteini/cgo/pull/9
[4]: https://aur.archlinux.org/cgit/aur.git/log/?h=fist
[5]: https://aur.archlinux.org/cgit/aur.git/log/?h=frotz-git
[6]: https://aur.archlinux.org/cgit/aur.git/log/?h=libbulletml
[7]: https://aur.archlinux.org/cgit/aur.git/log/?h=rrootage
[8]: https://aur.archlinux.org/cgit/aur.git/log/?h=translate-shell-git

Reply via email to