[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control

[bugzilla]

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

Christophe LACOMBE  changed:

   What|Removed |Added

  Flags|needinfo?(clacombe@amadeus. |
   |com)|



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control

[bugzilla]

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

Alexander Kurtakov akurt...@redhat.com changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

Christophe LACOMBE claco...@amadeus.com changed:

   What|Removed |Added

   Flag|needinfo?(claco...@amadeus. |
   |com)|

--- Comment #11 from Christophe LACOMBE claco...@amadeus.com 2010-11-10 
05:40:31 EST ---
Sorry for this absence.
I'm quiet busy on other topic for the time being, but should take care of it as
soon as possible.
Thanks.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

--- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-11-08 
12:43:04 EST ---
What is the status 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag||needinfo?(claco...@amadeus.
   ||com)

--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-09-22 
13:14:56 EDT ---
Christophe, ping?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

--- Comment #7 from Christophe LACOMBE claco...@amadeus.com 2010-09-02 
10:41:26 EDT ---
Thanks a lot Mamoru,
I'm working on the packaging of a new package stdinv (also required by stdair).
This package also requires extracc, but if I understand well, this is not an
issue as loong it is in review mode.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-09-02 
11:35:24 EDT ---
Okay, thank you.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

--- Comment #4 from Christophe LACOMBE claco...@amadeus.com 2010-09-01 
11:50:26 EDT ---
Dear Mamoru,

thanks a lot for the review, for your feedback... and for your support!

Yes, of course, I understand that I must bring more contribution than just that
package. If you do not mind, I will review other package as soon as I can.

In the mid-time, as extracc is used by other project (namely stdair
https://bugzilla.redhat.com/show_bug.cgi?id=614036), it is quite urgent to
deliver it to not block them. So, I have seen with Denis and Son and we propose
that Denis Arnaud do the request to submit extracc.

Then I will continue on my time to submit other review requests and do
pre-review to be sponsored later.

Any way, thanks again for that review: since it is my first step in the Fedora
packaging world, it is very particular for me; I'll remember for a long time :)

Best Regards

Christophe

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

--- Comment #5 from Denis Arnaud denis.arnaud_fed...@m4x.org 2010-09-01 
12:18:41 EDT ---
May I request the ownership for that package directly from here, or should I
create another review request linking to here?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

--- Comment #6 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-09-01 
13:32:33 EDT ---
@Christophe:
If you are going to give up importing this package for now,
when you submit another review request it will be the _first_ submission
of review request because this will be closed with NOTABUG (and
will be marked as a duplicate of the new review request if other
person submits new extracc review request). Is it okay?

Honestly saying, I don't want to close FE-NEEDSPONSER ticket
with NOTABUG easily.

@Denis:
(In reply to comment #5)
 May I request the ownership for that package directly from here, 
No. And note that I have _not approved_ this package yet.

 or should I create another review request linking to here?
Not linking here but mark this as a duplicate of the new one,
and you have to submit a srpm by yourself. However please wait
for the reply from Christophe.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review?

--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-08-23 
14:11:29 EDT ---
Okay, now:

-
NOTE: Before being sponsored:

This package will be accepted with another few (or no) work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to show that you have an understanding 
of the process and of the packaging guidelines as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a pre-review of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on my wiki page:
http://fedoraproject.org/wiki/User:Mtasaka#B._Review_request_tickets
(Check No one is reviewing)

Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets


-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

--- Comment #2 from Christophe LACOMBE claco...@amadeus.com 2010-08-22 
17:24:56 EDT ---
Thanks for that review. I have integrated your comments and generated a new
source RPM (corresponding to the new specification file), both available on
SourceForge:
Spec URL: http://downloads.sourceforge.net/extracc/extracc.spec
SRPM URL: http://downloads.sourceforge.net/extracc/extracc-0.5.0-2.fc13.src.rpm

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks||614036

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp

--- Comment #1 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-08-21 
13:39:24 EDT ---
Some notes:

* License
  - As far as I checked the source codes, the license tag should
be LGPLv2+.

* Version specific dependency
  - = 1.10 part on BuildRequires: cppunit-devel is not required
because cppunit-devel on currently supported Fedora / EPEL
branches all satisfy this version dependency. ref:
https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires

* pkgconfig dependency
  - Why does Requires: pkgconfig appear on -devel subpackage
although no pkgconfig .pc file is installed in -devel subpackage?

* Timestamps
  - Please consider to use

make install DESTDIR=$RPM_BUILD_ROOT INSTALL=install -p

to keep timestamps on installed files. This method usually works
for Makefiles generated by recent autotools.

* scriptlets
  - Current %post / %postun scriptlets for -devel subpackage are
just redundant and should be removed.

* Directory ownership issue
  - The directory %{_datadir}/%{name}/ itself is not owned by any
packages:
   
https://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership
https://fedoraproject.org/wiki/Packaging/UnownedDirectories#Common_Mistakes

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 616881] Review Request: extracc - Package to use C++ pojects with Cruise Control - FE-NEEDSPONSOR

[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=616881

Christophe LACOMBE claco...@amadeus.com changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review