[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#41010] Upgrade Oil to 0.8.pre4
From: |
Ryan Prior |
Subject: |
[bug#41010] Upgrade Oil to 0.8.pre4 |
Date: |
Sat, 02 May 2020 04:02:22 +0000 |
Tobias, thank you for the quick and thorough review! I've attached updated
patches.
A little context on how this patch came to be might make some of my choices a
little less mysterious: I didn't actually start with the original package
definition, because I didn't realize oil was already packaged in Guix. (I
searched for "osh", not "oil-shell")
So I wrote my own package, went to find where to put it in the guix source
tree, found the existing package, and then kinda merged them together.
On Friday, May 1, 2020 11:23 PM, Tobias Geerinckx-Rice <address@hidden> wrote:
> However, please do build and test patches before submitting them.
Done! I test my packages mostly by running `guix build -f mypackage.scm` so I
had to figure out how to build it in the context of the source tree.
> Add a full stop after commit summaries, and a ‘change log’ entry as commit
> body:
Added.
> OTOH a one-line link to that thread, if one exists, would be preferable.
Changed to a one-line link.
> Where do they recommend this? It's nice to have a link in case the
> recommendation changes.
Added a link to the recommendation.
> …as ‘guix gc --references oil’ (and readline's general nature) tell us,
> that's not the case here: Oil links to it at run time, so it must not be
> native.
I changed it to a normal input.
> Both the original synopsis and description are much better. If certain things
> are no longer accurate they can be adjusted but this is just upstream's
> marketing pitch.
I did ask upstream what they'd want as a synopsis/description. I updated it to
be more similar to the original but edited for accuracy.
>
> > - (license (list license:psfl ; Tarball vendors python2.7
>
> Hmm, this doesn't parse as English (it's missing a verb). I'd guess typo… but
> for what? Are upstream the ‘tarball vendors’ here? What was wrong with the
> original comment?
In some software development circles, we do use "vendor" as a verb. Sorry for
the confusion!
>vendor (third-person singular simple present vendors, present participle
>vendoring, simple past and past participle vendored)
>
> (transitive, software development) To bundle third-party dependencies with
> the source code for one's own program.
>
> I distributed my application with a vendored copy of Perl so that it
> wouldn't use the system copies of Perl where it is installed.
https://en.wiktionary.org/wiki/vendor
In favor of readability I changed the verb to "includes."
> Phew. ‘I'll just review this trivial bump before bed-time.’ This patch
> changes lots of things for one small package. I hope you don't mind lots of
> comments :-)
Thank you again, I appreciated all the comments!
Sincerely,
Ryan
0001-gnu-oil-Update-to-0.8.pre4.patch
Description: Text Data
0001-gnu-oil-shell-Rename-to-oil-and-add-a-deprecated-ali.patch
Description: Text Data
- [bug#41010] Rename & upgrade oil-shell, Ryan Prior, 2020/05/01
- [bug#41010] Upgrade Oil to 0.8.pre4, Ryan Prior, 2020/05/01
- [bug#41010] Upgrade Oil to 0.8.pre4, Tobias Geerinckx-Rice, 2020/05/01
- [bug#41010] Upgrade Oil to 0.8.pre4, Tobias Geerinckx-Rice, 2020/05/01
- [bug#41010] Upgrade Oil to 0.8.pre4,
Ryan Prior <=
- [bug#41010] Upgrade Oil to 0.8.pre4, Ryan Prior, 2020/05/15
- [bug#41010] Upgrade Oil to 0.8.pre4, Tobias Geerinckx-Rice, 2020/05/15
- [bug#41010] Upgrade Oil to 0.8.pre4, Tobias Geerinckx-Rice, 2020/05/16
- [bug#41010] Upgrade Oil to 0.8.pre4, Ryan Prior, 2020/05/17
- [bug#41010] Upgrade Oil to 0.8.pre4, Leo Famulari, 2020/05/02
[bug#41010] [PATCH 0/1] Updated patch for 0.8.pre5, Ryan Prior, 2020/05/28