unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#30665: ant-build-system: use manifest task
@ 2018-03-01 17:42 Ricardo Wurmus
  2018-06-18  8:31 ` bug#30665: [PATCH, STAGING] guix: ant-build-system: Use manifest task to create manifest Gábor Boskovits
  2018-06-18 15:18 ` bug#30665: ant-build-system: use-manifest-task Gábor Boskovits
  0 siblings, 2 replies; 4+ messages in thread
From: Ricardo Wurmus @ 2018-03-01 17:42 UTC (permalink / raw)
  To: 30665

Hi Guix,

instead of using a custom target that uses the “echo” task to write the
manifest, the ant-build-system should use the “manifest” task directly.

It is documented here:

    https://ant.apache.org/manual/Tasks/manifest.html

-- 
Ricardo

^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#30665: [PATCH, STAGING] guix: ant-build-system: Use manifest task to create manifest.
  2018-03-01 17:42 bug#30665: ant-build-system: use manifest task Ricardo Wurmus
@ 2018-06-18  8:31 ` Gábor Boskovits
  2018-06-18 13:49   ` Marius Bakke
  2018-06-18 15:18 ` bug#30665: ant-build-system: use-manifest-task Gábor Boskovits
  1 sibling, 1 reply; 4+ messages in thread
From: Gábor Boskovits @ 2018-06-18  8:31 UTC (permalink / raw)
  To: 30665

---
 guix/build/ant-build-system.scm | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/guix/build/ant-build-system.scm b/guix/build/ant-build-system.scm
index 3ed12b9f4..87c782d85 100644
--- a/guix/build/ant-build-system.scm
+++ b/guix/build/ant-build-system.scm
@@ -68,14 +68,11 @@
 
                  (target (@ (name "manifest"))
                          (mkdir (@ (dir "${manifest.dir}")))
-                         (echo (@ (file "${manifest.file}")
-                                  (message ,(string-append
-                                              (if main-class
-                                                (string-append
-                                                  "Main-Class: " main-class
-                                                  "${line.separator}")
-                                                "")
-                                              "")))))
+                         (manifest (@ (file "${manifest.file}"))
+                                   ,(if main-class
+                                       `(attribute (@ (name "Main-Class")
+                                                      (value ,main-class)))
+                                       "")))
 
                  (target (@ (name "compile"))
                          (mkdir (@ (dir "${classes.dir}")))
-- 
2.17.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* bug#30665: [PATCH, STAGING] guix: ant-build-system: Use manifest task to create manifest.
  2018-06-18  8:31 ` bug#30665: [PATCH, STAGING] guix: ant-build-system: Use manifest task to create manifest Gábor Boskovits
@ 2018-06-18 13:49   ` Marius Bakke
  0 siblings, 0 replies; 4+ messages in thread
From: Marius Bakke @ 2018-06-18 13:49 UTC (permalink / raw)
  To: Gábor Boskovits, 30665

[-- Attachment #1: Type: text/plain, Size: 1602 bytes --]

Gábor Boskovits <boskovits@gmail.com> writes:

> ---
>  guix/build/ant-build-system.scm | 13 +++++--------
>  1 file changed, 5 insertions(+), 8 deletions(-)
>
> diff --git a/guix/build/ant-build-system.scm b/guix/build/ant-build-system.scm
> index 3ed12b9f4..87c782d85 100644
> --- a/guix/build/ant-build-system.scm
> +++ b/guix/build/ant-build-system.scm
> @@ -68,14 +68,11 @@
>  
>                   (target (@ (name "manifest"))
>                           (mkdir (@ (dir "${manifest.dir}")))
> -                         (echo (@ (file "${manifest.file}")
> -                                  (message ,(string-append
> -                                              (if main-class
> -                                                (string-append
> -                                                  "Main-Class: " main-class
> -                                                  "${line.separator}")
> -                                                "")
> -                                              "")))))
> +                         (manifest (@ (file "${manifest.file}"))
> +                                   ,(if main-class
> +                                       `(attribute (@ (name "Main-Class")
> +                                                      (value ,main-class)))
> +                                       "")))

LGTM.  I was about to push it and start the 'staging' jobset on Hydra,
but I'm not sure what to write in the commit message.  Are you able to
push it soonish?

I'll start Hydra once the current 'master' evaluation finishes.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#30665: ant-build-system: use-manifest-task
  2018-03-01 17:42 bug#30665: ant-build-system: use manifest task Ricardo Wurmus
  2018-06-18  8:31 ` bug#30665: [PATCH, STAGING] guix: ant-build-system: Use manifest task to create manifest Gábor Boskovits
@ 2018-06-18 15:18 ` Gábor Boskovits
  1 sibling, 0 replies; 4+ messages in thread
From: Gábor Boskovits @ 2018-06-18 15:18 UTC (permalink / raw)
  To: 30665-done

[-- Attachment #1: Type: text/plain, Size: 68 bytes --]

Fixed on staging, commit:
90ea1006a86518e523c510eee1214fd81e71da74.

[-- Attachment #2: Type: text/html, Size: 103 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-06-18 15:19 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-03-01 17:42 bug#30665: ant-build-system: use manifest task Ricardo Wurmus
2018-06-18  8:31 ` bug#30665: [PATCH, STAGING] guix: ant-build-system: Use manifest task to create manifest Gábor Boskovits
2018-06-18 13:49   ` Marius Bakke
2018-06-18 15:18 ` bug#30665: ant-build-system: use-manifest-task Gábor Boskovits

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

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).