[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#26966: [PATCH 18/22] gnu: ant-bootstrap: Do not delete scripts.
From: |
Ricardo Wurmus |
Subject: |
bug#26966: [PATCH 18/22] gnu: ant-bootstrap: Do not delete scripts. |
Date: |
Thu, 18 May 2017 21:57:49 +0200 |
User-agent: |
mu4e 0.9.18; emacs 25.2.1 |
Roel Janssen <address@hidden> writes:
> Ricardo Wurmus writes:
>
>> The bootstrap ant should keep the scripts as their presence is checked when
>> building the latest version of ant.
>>
>> * gnu/packages/java.scm (ant-bootstrap)[arguments]: Delete "remove-scripts"
>> phase.
>> ---
>> gnu/packages/java.scm | 8 --------
>> 1 file changed, 8 deletions(-)
>>
>> diff --git a/gnu/packages/java.scm b/gnu/packages/java.scm
>> index c5fa67eef..2b9d8ddd4 100644
>> --- a/gnu/packages/java.scm
>> +++ b/gnu/packages/java.scm
>> @@ -210,14 +210,6 @@ This package provides the virtual machine.")
>> `(#:tests? #f ; no "check" target
>> #:phases
>> (modify-phases %standard-phases
>> - (add-after 'unpack 'remove-scripts
>> - ;; Remove bat / cmd scripts for DOS as well as the antRun and
>> runant
>> - ;; wrappers.
>> - (lambda _
>> - (for-each delete-file
>> - (find-files "src/script"
>> - "(.*\\.(bat|cmd)|runant.*|antRun.*)"))
>> - #t))
>> (delete 'configure)
>> (replace 'build
>> (lambda* (#:key inputs #:allow-other-keys)
>
> This seems redundant with the "04/22 gnu: Add ant-bootstrap" patch.
> Why add a snippet to remove it in a separate patch?
Oh, you’re right. These patches ought to be squashed into one.
“ant-bootstrap” was a mere copy of “ant” at this point, which did
include the “remove-scripts” phase.
I’ll fix this before pushing.
--
Ricardo
GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
https://elephly.net
- bug#26966: [PATCH 14/22] gnu: Add icedtea-6., (continued)
bug#26966: [PATCH 15/22] gnu: icedtea-7: Use icedtea-6 for bootstrapping., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 16/22] gnu: java-swt: Move below the bootstrap packages., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 17/22] gnu: clojure: Move below bootstrap packages., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 18/22] gnu: ant-bootstrap: Do not delete scripts., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 19/22] gnu: ant: Inherit from ant-bootstrap., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 20/22] gnu: ant: Delete bundled jars., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 21/22] gnu: ant: Update to 1.10.1., Ricardo Wurmus, 2017/05/17
bug#26966: [PATCH 22/22] gnu: Remove GCJ., Ricardo Wurmus, 2017/05/17