[Bug 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

2013-10-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1021164

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

Summary|Review Request: gpp -   |Review Request:
   |Customizable|general-purpose-preprocesso
   |language-agnostic   |r - Customizable
   |preprocessor|language-agnostic
   ||preprocessor



-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

2013-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1021164

Brendan Jones  changed:

   What|Removed |Added

 CC||brendan.jones...@gmail.com
   Assignee|nob...@fedoraproject.org|brendan.jones...@gmail.com
  Flags||fedora-review?



--- Comment #4 from Brendan Jones  ---
I'll take this on.

-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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

Brendan Jones  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Brendan Jones  ---
Rpmlint output is OK except this:

general-purpose-preprocessor-debuginfo.x86_64: E: description-line-too-long C
This package provides debug information for package
general-purpose-preprocessor.
general-purpose-preprocessor-debuginfo.x86_64: E: incorrect-fsf-address
/usr/src/debug/gpp-2.24/src/gpp.c

Fix the first and notify the author of the second at your discretion.

No other issues - this package is APPROVED


Review:

* Accompanying license is good and matches source headers
* Package doesn't create any directories it doesn't own
* SPEC file is according to standard and in legible US format
* Latest version is packaged
* Requires are sane
* Package functions as described

-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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



--- Comment #6 from Zbigniew Jędrzejewski-Szmek  ---
(In reply to Brendan Jones from comment #5)
> Rpmlint output is OK except this:
> 
> general-purpose-preprocessor-debuginfo.x86_64: E: description-line-too-long
> C This package provides debug information for package
> general-purpose-preprocessor.
The description is generated automatically. It's too long because of the long
package name... I think it's a known issue.

> general-purpose-preprocessor-debuginfo.x86_64: E: incorrect-fsf-address
> /usr/src/debug/gpp-2.24/src/gpp.c
> 
> Fix the first and notify the author of the second at your discretion.
Done. I took the opportunity to notify upstream maintainer about the package.

> No other issues - this package is APPROVED
Thank you for the 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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #7 from Zbigniew Jędrzejewski-Szmek  ---
New Package SCM Request
===
Package Name: general-purpose-preprocessor
Short Description: A general-purpose preprocessor with customizable syntax
Owners: zbyszek
Branches: f19 f20
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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



--- Comment #8 from Jon Ciesla  ---
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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



--- Comment #9 from Fedora Update System  ---
general-purpose-preprocessor-2.24-3.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/general-purpose-preprocessor-2.24-3.fc20

-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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

Fedora Update System  changed:

   What|Removed |Added

 Status|NEW |MODIFIED



-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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



--- Comment #10 from Fedora Update System  ---
general-purpose-preprocessor-2.24-3.fc19 has been submitted as an update for
Fedora 19.
https://admin.fedoraproject.org/updates/general-purpose-preprocessor-2.24-3.fc19

-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
Package general-purpose-preprocessor-2.24-3.fc20:
* should fix your issue,
* was pushed to the Fedora 20 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing
general-purpose-preprocessor-2.24-3.fc20'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-20024/general-purpose-preprocessor-2.24-3.fc20
then log in and leave karma (feedback).

-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||general-purpose-preprocesso
   ||r-2.24-3.fc19
 Resolution|--- |ERRATA
Last Closed||2013-11-06 22:31:19



--- Comment #12 from Fedora Update System  ---
general-purpose-preprocessor-2.24-3.fc19 has been pushed to the Fedora 19
stable repository.

-- 
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 1021164] Review Request: general-purpose-preprocessor - Customizable language-agnostic preprocessor

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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|general-purpose-preprocesso |general-purpose-preprocesso
   |r-2.24-3.fc19   |r-2.24-3.fc20



--- Comment #13 from Fedora Update System  ---
general-purpose-preprocessor-2.24-3.fc20 has been pushed to the Fedora 20
stable repository.

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