[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#31774] [PATCH 12/22] gnu: Add maven-core.
From: |
Julien Lepiller |
Subject: |
[bug#31774] [PATCH 12/22] gnu: Add maven-core. |
Date: |
Sun, 10 Jun 2018 20:46:35 +0200 |
Le Sun, 10 Jun 2018 19:16:43 +0200,
Danny Milosavljevic <address@hidden> a écrit :
> Hi Julien,
>
> On Sun, 10 Jun 2018 13:03:09 +0200
> Julien Lepiller <address@hidden> wrote:
>
> > + ;; FIXME: This should be taken care of by
> > plexus-component-metadata directly
>
> Is this possible now?
>
> > + (invoke "sh" "-c"
> > + (string-append
> > + "(cat
> > build/classes/META-INF/plexus/components.t.xml |"
> > + "sed -e 's|</component-set>||' -e
> > 's|</components>||' ; "
> > + "cat
> > src/main/resources/META-INF/plexus/artifact-handlers.xml |"
> > + " sed -e 's|<?xml.*||' -e
> > 's|<component-set>||' -e 's|<components>||'"
> > + " -e 's|</component-set>||' -e
> > 's|</components>||'; "
> > + "cat
> > src/main/resources/META-INF/plexus/components.xml |"
> > + " sed -e 's|<?xml.*||' -e
> > 's|<component-set>||' -e 's|<components>||'"
> > + " -e 's|</component-set>||' -e
> > 's|</components>||'; "
> > + "cat
> > src/main/resources/META-INF/plexus/default-bindings.xml |"
> > + " sed -e 's|<?xml.*||' -e
> > 's|<component-set>||' -e 's|<components>||' )>"
> > +
> > "build/classes/META-INF/plexus/components.xml"))
>
> XML isn't really line-based, so using (sxml simple) to merge would be
> more reliable. Error reporting would be better, too.
>
> https://www.gnu.org/software/guile/manual/html_node/Reading-and-Writing-XML.html#Reading-and-Writing-XML
So I tried to use that, and here is an updated patch. I had troubles
using match (it tried to evaluate the content of the list I wanted to
match with, although that doesn't happen at the REPL), so I used a
sequence of cdr/car instead. Thanks for the hint :)
0012-gnu-Add-maven-core.patch
Description: Text Data
- [bug#31774] [PATCH 06/22] gnu: Add java-plexus-cli., (continued)
- [bug#31774] [PATCH 06/22] gnu: Add java-plexus-cli., Julien Lepiller, 2018/06/10
- [bug#31774] [PATCH 08/22] gnu: Add maven-plugin-api., Julien Lepiller, 2018/06/10
- [bug#31774] [PATCH 10/22] gnu: Add maven-plugin-annotations., Julien Lepiller, 2018/06/10
- [bug#31774] [PATCH 11/22] gnu: Add java-plexus-component-metadata., Julien Lepiller, 2018/06/10
- [bug#31774] [PATCH 12/22] gnu: Add maven-core., Julien Lepiller, 2018/06/10
- [bug#31774] [PATCH 12/22] gnu: Add maven-core., Danny Milosavljevic, 2018/06/10
- [bug#31774] [PATCH 12/22] gnu: Add maven-core.,
Julien Lepiller <=
- [bug#31774] [PATCH 12/22] gnu: Add maven-core., Danny Milosavljevic, 2018/06/11
- [bug#31774] [PATCH 12/22] gnu: Add maven-core., Julien Lepiller, 2018/06/12
- [bug#31774] [PATCH 12/22] gnu: Add maven-core., Danny Milosavljevic, 2018/06/13
- [bug#31774] [PATCH 12/22] gnu: Add maven-core., Ludovic Courtès, 2018/06/14
- [bug#31774] [PATCH 12/22] gnu: Add maven-core., Danny Milosavljevic, 2018/06/13
[bug#31774] [PATCH 04/22] gnu: Add java-eclipse-jetty-webapp., Julien Lepiller, 2018/06/10
[bug#31774] [PATCH 07/22] gnu: Add java-qdox., Julien Lepiller, 2018/06/10
[bug#31774] [PATCH 13/22] gnu: Add maven-embedder., Julien Lepiller, 2018/06/10