[Bug 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

MartinKG  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #20 from MartinKG  ---
Package Change Request
==
Package Name: cxxtools
New Branches: fc23 devel
Owners: martinkg
InitialCC: 

[please untire cxxtools, because package vdr-lieve depends on it]

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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



--- Comment #21 from Jon Ciesla  ---
This SCM request method has been deprecated. Please see
https://fedoraproject.org/wiki/PackageDB_admin_requests.

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |



-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=821220

--- Comment #19 from Ville Skyttä ville.sky...@iki.fi ---
No need to reference this bug in updates, the package has been in the repos for
a long time already.

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

[Bug 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=821220

--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
cxxtools-2.2-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/cxxtools-2.2-1.fc18

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

[Bug 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=821220

--- Comment #18 from Fedora Update System upda...@fedoraproject.org ---
cxxtools-2.2-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/cxxtools-2.2-1.fc19

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

[Bug 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

MartinKG mgans...@alice.de changed:

   What|Removed |Added

Version|17  |rawhide
  Flags|fedora-cvs+ |
  Flags||fedora-cvs?

--- Comment #15 from MartinKG mgans...@alice.de ---
Package Change Request
==
Package Name: cxxtools
New Branches: f18
Owners: martinkg
InitialCC:

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #16 from Jon Ciesla limburg...@gmail.com ---
f18 branch exists.

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

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

   What|Removed |Added

 Resolution|NEXTRELEASE |ERRATA

--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
cxxtools-2.1.1-2.fc17 has been pushed to the Fedora 17 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org ---
cxxtools-2.1.1-2.fc16 has been pushed to the Fedora 16 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #12 from MartinKG mgans...@alice.de ---
cxxtools was pushed to testing
http://dl.fedoraproject.org/pub/fedora/linux/updates/testing/17/x86_64/

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #11 from Michael Schwendt mschwe...@gmail.com ---
Both updates for F17 and F16 are still pending, so they are not available and
won't be published ever. You need to request a push to testing.

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

MartinKG mgans...@alice.de changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2012-05-29 13:14:29

--- Comment #8 from MartinKG mgans...@alice.de ---
the package built successfully on koji

fc18-candidate:
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=4112594

f17-candidate: 
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=4112762

f16-candidate:
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=4112807

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
cxxtools-2.1.1-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/cxxtools-2.1.1-2.fc17

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
cxxtools-2.1.1-2.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/cxxtools-2.1.1-2.fc16

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

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

--- Comment #4 from Ville Skyttä ville.sky...@iki.fi ---
Looks fine otherwise, but there's a bundled MD5 implementation in src/md5.c
which looks like the Polstra one to me - if that's correct, should add:

   Provides: bundled(md5-polstra)

See https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries for more
info.

Approved - no need to submit a new package here just to fix this if you don't
want, but please be sure to fix it before the first build you submit to 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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

--- Comment #5 from MartinKG mgans...@alice.de ---
%changelog
* Mon May 28 2012 Martin Gansser linux4mar...@gmx.de - 2.1.1-2
- added Provides: bundled(md5-polstra)


New Package SCM Request
===
Package Name: cxxtools
Short Description: A collection of general-purpose C++ classes
Owners: martinkg
Branches: f16 f17
InitialCC:

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Ville Skyttä ville.sky...@iki.fi ---
Remember to set the fedora-cvs flag whenever making SCM requests...

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

MartinKG mgans...@alice.de changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|ville.sky...@iki.fi |nob...@fedoraproject.org

--- Comment #2 from MartinKG mgans...@alice.de ---
SRPM URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/cxxtools/cxxtools-2.1.1-1.fc17.src.rpm?a=w7DyN-YVNC0

Spec URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/cxxtools/cxxtools.spec?a=jFApyd75nmk

rpmlint output:

rpmlint cxxtools-2.1.1-1.fc17.src.rpm
cxxtools.src: I: enchant-dictionary-not-found en_US
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

rpmlint cxxtools-2.1.1-1.fc17.x86_64.rpm
cxxtools.x86_64: I: enchant-dictionary-not-found en_US
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

rpmlint cxxtools-debuginfo-2.1.1-1.fc17.x86_64.rpm
cxxtools-debuginfo.x86_64: I: enchant-dictionary-not-found en_US
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

rpmlint cxxtools-devel-2.1.1-1.fc17.x86_64.rpm
cxxtools-devel.x86_64: I: enchant-dictionary-not-found en_US
cxxtools-devel.x86_64: W: no-documentation
cxxtools-devel.x86_64: W: no-manual-page-for-binary cxxtools-config
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|ville.sky...@iki.fi

--- Comment #3 from Ville Skyttä ville.sky...@iki.fi ---
Will look into this later, but please don't touch the status or assigned to
fields between package revisions unless you want a new reviewer.

As noted in bug 819237 comment 9, please also re-fix your rpmlint/enchant
config and keep it fixed by installing the hunspell-en package.

One common and useful practice between package revisions is to post the
changelog after the previous release into the review bug.  There's no need to
repeatedly post the rpmlint output if it contains nothing of interest (such as
in comment 2).

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

 Blocks|177841  |
Summary|cxtools - A collection of   |cxxtools - A collection of
   |general-purpose C++ classes |general-purpose C++ classes

-- 
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 821220] cxxtools - A collection of general-purpose C++ classes

[bugzilla]

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

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ville.sky...@iki.fi
   Assignee|nob...@fedoraproject.org|ville.sky...@iki.fi
  Flags||fedora-review?

--- Comment #1 from Ville Skyttä ville.sky...@iki.fi ---
Looks like 2.1.1 is available.

Run unit tests during build, e.g. like this between %install and %clean:
%check
test/alltests

URL should probably be http://www.tntnet.org/cxxtools.html

Be consistent with the Group tag, either add one for the main package or remove
it from -devel too.

%{?_isa} missing from -devel's main package dep:
https://fedoraproject.org/wiki/Packaging/Guidelines#Requiring_Base_Package

 # FIXME: really? re-check!

Please re-check the license and remove the above comment when done.

 https://sourceforge.net/tracker/
 ?func=detailaid=3525604group_id=124828atid=700758

That's not a bug, just leave the file out from the package if it's empty.

-- 
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