[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2015-03-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks|652183 (FE-JAVASIG) |
 Resolution|--- |NOTABUG
Last Closed||2015-03-31 12:23:23



--- Comment #25 from gil cattaneo punto...@libero.it ---
No more interested


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker bug
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2014-07-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|mizde...@redhat.com |nob...@fedoraproject.org
  Flags|fedora-review?  |
   |needinfo?(mizdebsk@redhat.c |
   |om) |



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2014-07-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

   Priority|high|unspecified



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2014-07-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Christopher Meng i...@cicku.me changed:

   What|Removed |Added

  Flags||needinfo?(mizdebsk@redhat.c
   ||om)



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-11-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Bill Nottingham nott...@redhat.com changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
 CC|nott...@redhat.com  |



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-09-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

  Flags|needinfo?(mizdebsk@redhat.c |
   |om) |



--- Comment #24 from Mikolaj Izdebski mizde...@redhat.com ---
(In reply to Christopher Meng from comment #22)
 Ping?

What do you need to know?
If you want to take the review, feel free to do so.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=xLWyD0pLQOa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-09-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 CC||cicku...@gmail.com
  Flags||needinfo?(mizdebsk@redhat.c
   ||om)



--- Comment #22 from Christopher Meng cicku...@gmail.com ---
Ping?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=U0qHIQbQAza=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-09-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780



--- Comment #23 from gil cattaneo punto...@libero.it ---
pong?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=H8SeFA1guUa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-07-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #21 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-2.fc19.src.rpm

- removed POM modifications

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ZTdve4wDvaa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-07-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #20 from Mikolaj Izdebski mizde...@redhat.com ---
POM modifications just to remove warnings are missing the point.  If there is a
warning in the first place then the real reson should be fixed, not symptom. 
You can keep these modifications if you provide a reference to a bug report
(either upsteam of Fedora).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=erCG27aFq5a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #19 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-2.fc19.src.rpm

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=5553676

- Full XMvn support

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=6CFMLi73zOa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Bug 855780 depends on bug 977901, which changed state.

Bug 977901 Summary: java-service-wrapper: Wrong installation directory
https://bugzilla.redhat.com/show_bug.cgi?id=977901

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=RLgOvOkFFUa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Bug 855780 depends on bug 977904, which changed state.

Bug 977904 Summary: java-service-wrapper: Please install Maven POM file
https://bugzilla.redhat.com/show_bug.cgi?id=977904

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=H6ytzGYaPFa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #18 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-2.fc18.src.rpm

- replace mvn-rpmbuild with mvn_build

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=soUPYQhKORa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #9 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-2.fc18.src.rpm

- minor changes for the new guideline

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=q8qrznmgTPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #10 from Mikolaj Izdebski mizde...@redhat.com ---
There are several problems with this package.

First of all, the package is noarch, but depends on arch-specific packages and
hardcodes paths to libraries. This means that if this package is built on
32-bit system then it won't work properly on 64-bit ones and vice-versa.

You are using Maven scope system. What's the reason behind that? Maven should
be able to locate artifact in /usr/lib too. If not then either there is a bug
in the package providing the artifact or in XMvn resolver. In either case this
should be reported and fixed. Workaround is acceptable only if the real issue
was reported.

You are injecting versionany/version to some POMs. What's the reason?
Maven should be able to work without this. If it does not, please report a bug.

You are using old packaging guidelines. Please use the new guidelines for
Fedora 19+. Fedora 19 is soon to be released. After Fedora 19 GA no new
packages should be added to Fedora 18, so there is no reason to retain
backwards-compatibility.

 %global bits   %(rpm --eval %{__isa_bits}  | sed 's/32//')
 #%%global libdir %%{_prefix}/lib%%{bits}

That's not needed (see above).

 # svn export 
 http://svn.apache.org/repos/asf/directory/deceased/daemon/tags/daemon-parent-1.1.8/
  apacheds-daemon-1.1.8
 # rm -rf 
 apacheds-daemon-1.1.8/plugin/src/main/resources/org/apache/directory/daemon/installers/*
 # find apacheds-daemon-1.1.8 -name '*.bat' -delete
 # find apacheds-daemon-1.1.8 -name '*.class' -delete
 # find apacheds-daemon-1.1.8 -name '*.dll' -delete
 # find apacheds-daemon-1.1.8 -name '*.exe' -delete
 # find apacheds-daemon-1.1.8 -name '*.jar' -type f -delete
 # find apacheds-daemon-1.1.8 -name '*.jnilib' -delete
 # find apacheds-daemon-1.1.8 -name 'jsvc_*' -delete
 # find apacheds-daemon-1.1.8 -name '*.so' -delete
 # find apacheds-daemon-1.1.8 -name 'wrapper-*' -type f -delete
 # tar czf apacheds-daemon-1.1.8-clean-src-svn.tar.gz apacheds-daemon-1.1.8
 Source0:   %{name}-%{version}-clean-src-svn.tar.gz

To keep spec file simpler it would be better to create a separate shell script
(cleanup-tarball.sh) to do the cleaning.

 Patch0:%{name}-%{version}-disable-izpack.patch

I don't know what is the purpose of this patch. Every patch should have link to
upstream bug or a comment explaining its purpose.

 BuildRequires: java-devel

Not needed.

 sed -i 's|${version}|${project.version}|' pom.xml

Is this necessary? Either explain why or provide link to upstream bug tracker.

 %pom_xpath_inject 
 pom:project/pom:reporting/pom:plugins/pom:plugin[pom:artifactId='maven-surefire-report-plugin']
  
 versionany/version

I don't see why would you inject version any to the POM. Again, you need to
explain why you are doing this.

 %pom_remove_dep tanukisoft:wrapper
 %pom_xpath_inject pom:project/pom:dependencyManagement/pom:dependencies '
 dependency
   groupIdtanukisoft/groupId
   artifactIdwrapper/artifactId
   versionany/version
   scopesystem/scope
   
 systemPath%{_prefix}/lib%{bits}/java-service-wrapper/wrapper.jar/systemPath
 /dependency'
 %pom_xpath_inject pom:project/pom:dependencies '
 dependency
   groupIdtanukisoft/groupId
   artifactIdwrapper/artifactId
   versionany/version
   scopesystem/scope
   
 systemPath%{_prefix}/lib%{bits}/java-service-wrapper/wrapper.jar/systemPath
 /dependency'

There is no need to use scope system with systemPath. Maven should be able to
locate all artifacts, also under /usr/lib.

 %pom_remove_dep org.apache.maven:maven-project plugin
 %pom_add_dep org.apache.maven:maven-core plugin

I understand why this was done (Maven 2 - Maven 3 migration), but that doesn't
have to be obvious for everyone. Please add a comment.

 %pom_remove_dep plexus:plexus-utils plugin
 %pom_add_dep org.codehaus.plexus:plexus-utils plugin

plexus:plexus-utils is available in Fedora. There is no need to replace it with
org.codehaus.plexus:plexus-utils.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=h6SNYgMYrNa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #11 from gil cattaneo punto...@libero.it ---
(In reply to Mikolaj Izdebski from comment #10)
 
  %global bits   %(rpm --eval %{__isa_bits}  | sed 's/32//')
  #%%global libdir %%{_prefix}/lib%%{bits}
 
 That's not needed (see above).
 
  # svn export 
  http://svn.apache.org/repos/asf/directory/deceased/daemon/tags/daemon-parent-1.1.8/
   apacheds-daemon-1.1.8
  # rm -rf 
  apacheds-daemon-1.1.8/plugin/src/main/resources/org/apache/directory/daemon/installers/*
  # find apacheds-daemon-1.1.8 -name '*.bat' -delete
  # find apacheds-daemon-1.1.8 -name '*.class' -delete
  # find apacheds-daemon-1.1.8 -name '*.dll' -delete
  # find apacheds-daemon-1.1.8 -name '*.exe' -delete
  # find apacheds-daemon-1.1.8 -name '*.jar' -type f -delete
  # find apacheds-daemon-1.1.8 -name '*.jnilib' -delete
  # find apacheds-daemon-1.1.8 -name 'jsvc_*' -delete
  # find apacheds-daemon-1.1.8 -name '*.so' -delete
  # find apacheds-daemon-1.1.8 -name 'wrapper-*' -type f -delete
  # tar czf apacheds-daemon-1.1.8-clean-src-svn.tar.gz apacheds-daemon-1.1.8
  Source0:   %{name}-%{version}-clean-src-svn.tar.gz
 
 To keep spec file simpler it would be better to create a separate shell
 script (cleanup-tarball.sh) to do the cleaning.
 
  Patch0:%{name}-%{version}-disable-izpack.patch
 
 I don't know what is the purpose of this patch. Every patch should have link
 to upstream bug or a comment explaining its purpose.
 
  BuildRequires: java-devel
 
 Not needed.
ok removed
  sed -i 's|${version}|${project.version}|' pom.xml
 
 Is this necessary? Either explain why or provide link to upstream bug
 tracker.
 
  %pom_xpath_inject 
  pom:project/pom:reporting/pom:plugins/pom:plugin[pom:artifactId='maven-surefire-report-plugin']
   
  versionany/version
 
the intent is to remove all warnings

 I don't see why would you inject version any to the POM. Again, you need
 to explain why you are doing this.
 
  %pom_remove_dep tanukisoft:wrapper
  %pom_xpath_inject pom:project/pom:dependencyManagement/pom:dependencies '
  dependency
groupIdtanukisoft/groupId
artifactIdwrapper/artifactId
versionany/version
scopesystem/scope

  systemPath%{_prefix}/lib%{bits}/java-service-wrapper/wrapper.jar/systemPath
  /dependency'
  %pom_xpath_inject pom:project/pom:dependencies '
  dependency
groupIdtanukisoft/groupId
artifactIdwrapper/artifactId
versionany/version
scopesystem/scope

  systemPath%{_prefix}/lib%{bits}/java-service-wrapper/wrapper.jar/systemPath
  /dependency'
 
 There is no need to use scope system with systemPath. Maven should be able
 to locate all artifacts, also under /usr/lib.
 

java-service-wrapper don provides maven depmap or pom ... maven or xmvn work
also without these file?

  %pom_remove_dep org.apache.maven:maven-project plugin
  %pom_add_dep org.apache.maven:maven-core plugin
 
 I understand why this was done (), but that
 doesn't have to be obvious for everyone. Please add a comment.
 
ok done
  %pom_remove_dep plexus:plexus-utils plugin
  %pom_add_dep org.codehaus.plexus:plexus-utils plugin
 
 plexus:plexus-utils is available in Fedora. There is no need to replace it
 with org.codehaus.plexus:plexus-utils.
ok removed

Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-2.fc18.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=2bqxjQAZz8a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #12 from Mikolaj Izdebski mizde...@redhat.com ---
(In reply to gil cattaneo from comment #11)
   %pom_xpath_inject 
   pom:project/pom:reporting/pom:plugins/pom:plugin[pom:artifactId='maven-surefire-report-plugin']

   versionany/version
  
 the intent is to remove all warnings

There should be no warnings in Fedora 19+.  If there are please file a bug
report against xmvn package.

  There is no need to use scope system with systemPath. Maven should be able
  to locate all artifacts, also under /usr/lib.
  
 
 java-service-wrapper don provides maven depmap or pom ... maven or xmvn work
 also without these file?

Then these files should be added to java-service-wrapper package.

 Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
 SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-2.fc18.src.rpm

The most important issues are still open.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=22bdMImbqya=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Depends On||977901

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=wnpcOjFwbYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Depends On||977904

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=XYUFdwTl3ra=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #13 from Mikolaj Izdebski mizde...@redhat.com ---
I have filled 2 bugs for java-service-wrapper package.  That's what should be
done instead of adding ugly hacks like scope system and systemPath...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=zvfbmRhrJka=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #14 from Mikolaj Izdebski mizde...@redhat.com ---
java-service-wrapper is fixed in rawhide now, you should remove your
workarounds.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=7cReHAXddsa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #15 from gil cattaneo punto...@libero.it ---
Great! thanks

Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-2.fc18.src.rpm

- removed java-service-wrapper workarounds

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=2jS9m8H13Xa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #16 from gil cattaneo punto...@libero.it ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=5539859

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=hir8ifd72sa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #17 from Mikolaj Izdebski mizde...@redhat.com ---
Please convert the package to current packaging guidelines.  This means using
%mvn_build instead of mvn-rpmbuild.  mvn-rpmbuild is a legacy feature and it
shouldn't be used for new packages.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=N2xzmvpHVBa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2013-06-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

   Priority|medium  |high

--- Comment #8 from Mikolaj Izdebski mizde...@redhat.com ---
Somehow I missed this bug, sorry for the delay.
I'll try to review this package ASAP.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=oB1G48Z2VAa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-10-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #7 from Mikolaj Izdebski mizde...@redhat.com ---
As it's now the package won't work peoperly, the problem is libdir usage in
noarch package.

If package is built on 64-bit system then systemPath in pom.xml file will
contain reference to /usr/lib64 and this package will work only on 64-bit
systems (and vice versa, if the package is compiled on 32-bit system it will
only work on 32-bit systems).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-10-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #6 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/apacheds-daemon/1/apacheds-daemon.spec
SRPM URL:
http://gil.fedorapeople.org/apacheds-daemon/1/apacheds-daemon-1.1.8-2.fc16.src.rpm

- fix build problems x86_64 arch

tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=4577842

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-10-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-review?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-10-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

  Flags||needinfo?(puntogil@libero.i
   ||t)

--- Comment #2 from Mikolaj Izdebski mizde...@redhat.com ---
Fails to build in rawhide mock:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4574340

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-10-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #3 from Mikolaj Izdebski mizde...@redhat.com ---
This package depends on tanukisoft:wrapper, but currently nothing in fedora
provides this artifact.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-10-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

  Flags|needinfo?(puntogil@libero.i |
   |t)  |

--- Comment #4 from gil cattaneo punto...@libero.it ---
build fine also with java-service-wrapper
but i dont know how fix the problem if use x86_64 arch in koji

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-10-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

--- Comment #5 from Mikolaj Izdebski mizde...@redhat.com ---
(In reply to comment #4)
 build fine also with java-service-wrapper
 but i dont know how fix the problem if use x86_64 arch in koji

This will definitely have to be resolved.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-09-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mizde...@redhat.com
   Assignee|nob...@fedoraproject.org|mizde...@redhat.com

--- Comment #1 from Mikolaj Izdebski mizde...@redhat.com ---
I am taking this review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-09-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Blocks||823967

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review