GNU bug report logs - #57726
[PATCH] gnu: Add java-eclipse-sisu-maven-plugin.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 57726 in the body.
You can then email your comments to 57726 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: "Artyom V. Poptsov" <poptsov.artyom <at> gmail.com>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: Add java-eclipse-sisu-maven-plugin.
Date: Sun, 11 Sep 2022 12:11:23 +0300

[Message part 1 (text/plain, inline)]

Hello,

this patch adds "java-eclipse-sisu-maven-plugin". [1]

[0001-gnu-Add-java-eclipse-sisu-maven-plugin.patch (text/x-diff, attachment)]

[Message part 3 (text/plain, inline)]

The patch requires "slf4j-nop" package:
  https://debbugs.gnu.org/cgi/bugreport.cgi?bug=57724

Thanks,
- Artyom

References:
1. https://github.com/eclipse/sisu.mojos

-- 
Artyom "avp" Poptsov <poptsov.artyom <at> gmail.com>
Home page: https://memory-heap.org/~avp/
CADR Hackerspace co-founder: https://cadrspace.ru/
GPG: D0C2 EAC1 3310 822D 98DE  B57C E9C5 A2D9 0898 A02F

[signature.asc (application/pgp-signature, inline)]

Message #8 received at 57726 <at> debbugs.gnu.org (full text, mbox):

From: Julien Lepiller <julien <at> lepiller.eu>
To: "Artyom V. Poptsov" <poptsov.artyom <at> gmail.com>, 57726 <at> debbugs.gnu.org
Subject: Re: [bug#57726] [PATCH] gnu: Add java-eclipse-sisu-maven-plugin.
Date: Sun, 11 Sep 2022 12:05:06 +0200

[Message part 1 (text/plain, inline)]

The name implies this is a plugin for maven. The way you build it, maven will not be able to use it because it's missing some metadata.

You can use generate-plugin.xml to generate the required file. The parser might not work properly, I'll have a look if it doesn't work, but that might take some time.

Le 11 septembre 2022 11:11:23 GMT+02:00, "Artyom V. Poptsov" <poptsov.artyom <at> gmail.com> a écrit :
>Hello,
>
>this patch adds "java-eclipse-sisu-maven-plugin". [1]

[Message part 2 (text/html, inline)]

Message #11 received at 57726 <at> debbugs.gnu.org (full text, mbox):

From: "Artyom V. Poptsov" <poptsov.artyom <at> gmail.com>
To: Julien Lepiller <julien <at> lepiller.eu>
Cc: 57726 <at> debbugs.gnu.org
Subject: Re: [bug#57726] [PATCH] gnu: Add java-eclipse-sisu-maven-plugin.
Date: Sat, 17 Sep 2022 00:28:00 +0300

[Message part 1 (text/plain, inline)]

Hello Julien.

> The name implies this is a plugin for maven. The way you build it,
> maven will not be able to use it because it's missing some metadata.

Okay.  I moved the package to (gnu packages maven) as it seems to be
the more appropriate place for Maven plugins indeed.

> You can use generate-plugin.xml to generate the required file. The
> parser might not work properly, I'll have a look if it doesn't work, but
> that might take some time.

Thanks for the hint, I modified the patch so now the package uses
"generate-plugin.xml".

Please find the new patch attached.

[0001-gnu-Add-maven-sisu-plugin.patch (text/x-diff, attachment)]

[Message part 3 (text/plain, inline)]

- Artyom

-- 
Artyom "avp" Poptsov <poptsov.artyom <at> gmail.com>
Home page: https://memory-heap.org/~avp/
CADR Hackerspace co-founder: https://cadrspace.ru/
GPG: D0C2 EAC1 3310 822D 98DE  B57C E9C5 A2D9 0898 A02F

[signature.asc (application/pgp-signature, inline)]

Message #14 received at 57726 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: "Artyom V. Poptsov" <poptsov.artyom <at> gmail.com>
Cc: Julien Lepiller <julien <at> lepiller.eu>, 57726 <at> debbugs.gnu.org
Subject: Re: bug#57726: [PATCH] gnu: Add java-eclipse-sisu-maven-plugin.
Date: Sat, 24 Sep 2022 15:52:56 +0200

Hi!

Julien, WDYT?  :-)

Ludo’.

"Artyom V. Poptsov" <poptsov.artyom <at> gmail.com> skribis:

> Hello Julien.
>
>> The name implies this is a plugin for maven. The way you build it,
>> maven will not be able to use it because it's missing some metadata.
>
> Okay.  I moved the package to (gnu packages maven) as it seems to be
> the more appropriate place for Maven plugins indeed.
>
>> You can use generate-plugin.xml to generate the required file. The
>> parser might not work properly, I'll have a look if it doesn't work, but
>> that might take some time.
>
> Thanks for the hint, I modified the patch so now the package uses
> "generate-plugin.xml".
>
> Please find the new patch attached.
>
>
>
> - Artyom

Message #19 received at 57726-done <at> debbugs.gnu.org (full text, mbox):

From: Julien Lepiller <julien <at> lepiller.eu>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: "Artyom V. Poptsov" <poptsov.artyom <at> gmail.com>, 57726-done <at> debbugs.gnu.org
Subject: Re: bug#57726: [PATCH] gnu: Add java-eclipse-sisu-maven-plugin.
Date: Sat, 24 Sep 2022 19:07:02 +0200

Le Sat, 24 Sep 2022 15:52:56 +0200,
Ludovic Courtès <ludo <at> gnu.org> a écrit :

> Hi!
> 
> Julien, WDYT?  :-)
> 

Looks like I didn't receive the reply. The patch is now pushed to
master as fc07082fe3d750c85282c4aa57d01a36ebe9296b, thanks!

Again, I noticed qa.guix.gnu.org doing its work :)

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.