Your message dated Wed, 14 Aug 2013 21:04:47 +0000
with message-id <[email protected]>
and subject line Bug#712658: fixed in maven-repo-helper 1.8.4
has caused the Debian Bug report #712658,
regarding maven-repo-helper: POMReader doesn't trim whitespace
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
712658: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=712658
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: maven-repo-helper
Severity: important
Tags: patch

Because of recent changes in POMReader.java, values parsed from XML keep their
leading and trailing whitespace.

Consequently, a malformed .mh/pom.properties is created for example from 
pom.xml[1],
causing a build failure.

I propose a very simple patch to fix this issue.

Regards,

Jakub

[1] 
http://anonscm.debian.org/viewvc/pkg-java/trunk/bnd/debian/pom.xml?revision=15264&view=markup
>From 784c41c79cfa7d1ca660a2a2a4237b88bb043a54 Mon Sep 17 00:00:00 2001
From: Jakub Adam <[email protected]>
Date: Tue, 18 Jun 2013 12:54:23 +0200
Subject: [PATCH] Trim whitespace from values in POMReader

---
 src/main/java/org/debian/maven/repo/POMReader.java |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/debian/maven/repo/POMReader.java b/src/main/java/org/debian/maven/repo/POMReader.java
index cc2f4ce..47b2408 100644
--- a/src/main/java/org/debian/maven/repo/POMReader.java
+++ b/src/main/java/org/debian/maven/repo/POMReader.java
@@ -103,7 +103,7 @@ public class POMReader {
                 }
 
                 case XMLStreamConstants.END_ELEMENT: {
-                    String value = buffer != null ? buffer.toString() : null;
+                    String value = buffer != null ? buffer.toString().trim() : null;
                     if (inIgnoredElement > 0 || path.contains("exclusions")) {
                         // ignore
                     } else if (path.contains("dependency") || path.contains("plugin") || path.contains("extension")) {
-- 
1.7.10.4


--- End Message ---
--- Begin Message ---
Source: maven-repo-helper
Source-Version: 1.8.4

We believe that the bug you reported is fixed in the latest version of
maven-repo-helper, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Emmanuel Bourg <[email protected]> (supplier of updated maven-repo-helper 
package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Wed, 07 Aug 2013 17:53:13 +0200
Source: maven-repo-helper
Binary: maven-repo-helper
Architecture: source all
Version: 1.8.4
Distribution: unstable
Urgency: low
Maintainer: Debian Java Maintainers 
<[email protected]>
Changed-By: Emmanuel Bourg <[email protected]>
Description: 
 maven-repo-helper - Helper tools for including Maven metadata in Debian 
packages
Closes: 712658
Changes: 
 maven-repo-helper (1.8.4) unstable; urgency=low
 .
   * Fixed a regression in the pom parser (Closes: #712658)
   * The name of the pom is now displayed when a parsing error occurs
Checksums-Sha1: 
 93a36f574c4016d371ac819249bc8c7998eaea5f 1988 maven-repo-helper_1.8.4.dsc
 135c2edc8fb74bf173d48e4ba6951b5d16b5c7b2 158645 maven-repo-helper_1.8.4.tar.gz
 cceeb1a29e3ca177258afb8a348506b015aea006 130324 maven-repo-helper_1.8.4_all.deb
Checksums-Sha256: 
 a319e21afbfbae1d3fa5006088fdff7a7aeeac8c15b3b6d548fb9f87324bef96 1988 
maven-repo-helper_1.8.4.dsc
 e537d9b6a9843bd52f208e61ecdb6908921438d5023e141376a18cb2224684a6 158645 
maven-repo-helper_1.8.4.tar.gz
 b7da34eb1d636824b5128cbb2100e5c50de27139323f3c616de70ebf7ac65539 130324 
maven-repo-helper_1.8.4_all.deb
Files: 
 4149c6991bfa4f8a08223c33b81ad98c 1988 java optional maven-repo-helper_1.8.4.dsc
 fc1f3558feb4463d6e115b4dfc9e296e 158645 java optional 
maven-repo-helper_1.8.4.tar.gz
 ee20dacfd0101345fca6458a34124677 130324 java optional 
maven-repo-helper_1.8.4_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.14 (GNU/Linux)

iQIcBAEBCAAGBQJSC+9+AAoJEL/srsug59jDvqkP/j5/KgJan7rBNctdbuodWMnp
K04j+eortIR2eXmJNoa6L7nSAxbXeIRPZ2UyOtyrB4R08LDhQJDhxPJzN5fY7zt5
z/ziHO08W5XxlaJk2rD6YS5+jn6VLBYpYygP2dSqrLVz/ezaIPUZkwzyOg1BUA93
lyy0d++/u5kRaxIQeLScfm81htzwFdkWj0SMF1IpTaT6WtO+Wk+DLa/gLNfVRrqN
UPX8XOCJ2ZzMHRBjxZZ+wvYOWJl+fSS/O1QluEHa2OZv+Y3F3BvahTCJ2CiFK+7L
szNnnxS90kfLCkop0EG7jTYLUTxwjlM6msQbJh6WF5ND9c2PuECk0DlhF0V429lr
Tx6PrxZkVPL7l9UEP6Cp661grRCW+WnVpH6ZKmA0Rqy14/GaE2li/4WPWnd8bQFd
25dOOK0HR4xp52Xt9eksZmi8mzvR+btELMtsBpPSOD3s/QVbKiWZXYE6vOygXM9o
QxL4wmice4E/RLO7iwhCwW6MXkOE2QwGXm3GP9DGPNXU+r8B9R7lsXNXhNW1rkxx
v2raWojK6iFIxjOilXZ7L2RGkZaUw6YW8hVhTPNtGduvrD1MNE/mFYDjzZqHhsju
QA2O46JPmOOv1uKWlOwrcbVW0LVEEiabFWCevZ3sWL51Xypn6x2TC4ou6BFP0a3+
0WUvhVNiGWqzZKx58dW8
=YH4I
-----END PGP SIGNATURE-----

--- End Message ---
__
This is the maintainer address of Debian's Java team
<http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-java-maintainers>. 
Please use
[email protected] for discussions and questions.

Reply via email to