[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-12-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||1.3-6.fc12
 Resolution||ERRATA




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-12-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-12-24 15:42:40 EDT ---
picocontainer-1.3-6.fc12 has been pushed to the Fedora 12 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-12-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-12-08 03:04:49 EDT ---
picocontainer-1.3-6.fc12 has been pushed to the Fedora 12 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update picocontainer'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F12/FEDORA-2009-12926

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-12-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-12-06 18:59:13 EDT ---
picocontainer-1.3-6.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/picocontainer-1.3-6.fc12

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

  Status Whiteboard||F13FeatureIDEA




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:44:52 EDT 
---
CVS done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Michal Ingeli m...@v3.sk changed:

   What|Removed |Added

 CC||vivek...@gmail.com




--- Comment #4 from Michal Ingeli m...@v3.sk  2009-11-15 05:55:30 EDT ---
*** Bug 252119 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag|fedora-review?  |fedora-review+




--- Comment #5 from Lubomir Rintel lkund...@v3.sk  2009-11-15 06:17:32 EDT ---
Looks good now.
APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Michal Ingeli m...@v3.sk changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Michal Ingeli m...@v3.sk  2009-11-15 07:19:22 EDT ---
New Package CVS Request
===
Package Name: picocontainer
Short Description: Dependency-injection container
Owners: ksyz
Branches: F-11 F-12 EL-5
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585





--- Comment #1 from Michal Ingeli m...@v3.sk  2009-11-14 13:17:22 EDT ---
Builds in mock and koji [1], rpmlint quiet.

[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=1806166

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585


Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 CC||lkund...@v3.sk
 AssignedTo|nob...@fedoraproject.org|lkund...@v3.sk
   Flag||fedora-review?




--- Comment #2 from Lubomir Rintel lkund...@v3.sk  2009-11-14 16:39:41 EDT ---
1.) BuildRoot, please fix it up according to guidelines.

BuildRoot:  %{_tmppath}/%{name}-%{version}-%{release}-root

2.) No useless comments please

#setup -q -c -n %{name}-%{version}

3.) Please don't do the jpackage javadoc magic:

ln -s %{name}-%{version} $RPM_BUILD_ROOT%{_javadocdir}/%{name}
%ghost %doc %{_javadocdir}/%{name}

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537585





--- Comment #3 from Michal Ingeli m...@v3.sk  2009-11-14 21:22:28 EDT ---
(In reply to comment #2)
 1.) BuildRoot, please fix it up according to guidelines.
 
 BuildRoot:  %{_tmppath}/%{name}-%{version}-%{release}-root

Looking at [1], buildroot seems to be rather valid and according to guidelines.

 2.) No useless comments please
 
 #setup -q -c -n %{name}-%{version}

Right, removed that one.

 3.) Please don't do the jpackage javadoc magic:
 
 ln -s %{name}-%{version} $RPM_BUILD_ROOT%{_javadocdir}/%{name}
 %ghost %doc %{_javadocdir}/%{name}  

Removed.

Also decided to removed -target 1.5 and -encoding param for javac as it is may
be better not force thing. And forgotten -verbose.

Spec URL: http://v3.sk/~xyzz/rpm/intellij-idea/picocontainer.spec
SRPM URL: http://v3.sk/~xyzz/rpm/intellij-idea/picocontainer-1.3-6.fc11.src.rpm

[1] http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review