[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora --- Additional Comments From [EMAIL PROTECTED] 2007-10-27 10:41 EST --- Package Change Request == Package Name: tre New Branches: EL-5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-08-04 15:32 EST --- Package imported and devel build successful. FC-4 and FC-5 branches requested. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2006-08-04 18:51 EST --- FC-4 and FC-5 built, closing as NEXTRELEASE. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-08-03 11:52 EST --- http://rpm.greysector.net/extras/tre.spec http://rpm.greysector.net/extras/tre-0.7.4-3.src.rpm Fixed. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-08-03 22:55 EST --- Everything looks fixed up and therefore this package is APPROVED. Don't forget to close this bug as NEXTRELEASE once it's been imported and built. Also, don't forget to submit a bug against the package once it's got a bugzilla component about the breakage on x64. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-08-02 03:33 EST --- Well, I'm in no position to complain about delays, so no worries. ;) Thanks for taking the time. I'll try reporting the crash this week. Actually, it's crm114 that triggers the crash. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: tre - POSIX compatible regexp library with approximate
matching
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609
--- Additional Comments From [EMAIL PROTECTED] 2006-08-02 14:51 EST ---
ok, here's a review:
OK - Package name
OK - Spec file matches base package name.
OK - Meets Packaging Guidelines.
OK - License (LGPL)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
8b4bfb078f2cc9e01f37d3d251672f75 tre-0.7.4.tar.bz2
8b4bfb078f2cc9e01f37d3d251672f75 tre-0.7.4.tar.bz2.1
OK - Package compiles and builds on at least one arch.
See below - Package needs ExcludeArch
OK - BuildRequires correct
OK - Spec handles locales/find_lang
OK - Spec has needed ldconfig in post and postun
n/a - Package is relocatable and has a reason to be.
OK - Package owns all the directories it creates.
OK - Package has no duplicate files in %files.
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Spec has consistant macro usage.
OK - Package is code or permissible content.
n/a - -doc subpackage needed/used.
OK - Packages %doc files don't affect runtime.
OK - Headers/static libs in -devel subpackage.
OK - .pc files in -devel subpackage.
OK - .so files in -devel subpackage.
See below - -devel package Requires: %{name} = %{version}-%{release}
OK - .la files are removed.
n/a - Package is a GUI app and has a .desktop file
OK - Package doesn't own any directories other packages own.
See below - No rpmlint output.
Issues:
1. Not a blocker, but you might consider using a dist tag:
http://fedoraproject.org/wiki/Packaging/DistTag
2. The devel package should Requires: %{name} = %{version}-%{release}
3. rpmlint has some output: (all warnings, but should be cleaned up)
W: agrep summary-ended-with-dot Approximate grep utility.
W: tre macro-in-%changelog doc
W: tre macro-in-%changelog post
W: tre mixed-use-of-spaces-and-tabs
W: tre-devel summary-ended-with-dot Development files for use with the tre
package.
W: tre-devel no-documentation
(the last one should be ignored)
4. You mention crashes with the x86_64 version. Perhaps
the package should be ExcludeArch'ed for now until the bug is
found/fixed? I don't have a x86_64 or ppc test machine here, perhaps
someone else would like to give it a try on those arches before
approval?
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-08-02 17:42 EST --- http://rpm.greysector.net/extras/tre.spec http://rpm.greysector.net/extras/tre-0.7.4-2.src.rpm Fixed all of the above. Thanks for the review! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-07-28 15:06 EST --- http://rpm.greysector.net/extras/tre.spec http://rpm.greysector.net/extras/tre-0.7.4-1.src.rpm But I'm afraid the 64bit issues are still there, i.e. it still crashes on me. Also, the license changed to LGPL. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2006-07-25 19:14 EST --- Adding myself to CC list due to above-mentioned interest in crm114 (for a LUG colleague). It appears that tre 0.7.4 (released a couple months ago) may resolve the issues with x86_64 which caused the original delays with this package moving forward (which I learned of by discussing this package with Dominik on IRC). Alas, I lack access to an x86_64 buildhost (well, for scratch builds), so I can't verify this. If Dominik doesn't have the time to continue this (which was mentioned), I'd be happy to pick up the pieces he put together and maintain them. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-06-29 15:49 EST --- Any further progress on an updated submission? There is interest in the crm114 submission that requires this package... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-06-16 18:32 EST --- Ping Dominik: Are you still interested in packaging this? If I don't hear back from you in a bit more, I will close this review request. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 --- Additional Comments From [EMAIL PROTECTED] 2006-06-16 18:56 EST --- Still here, sorry for the long silence. I'll try to update all of my requests over the weekend. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778, 187610 |177841 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-05-29 07:18 EST --- Dropping FE-NEEDSPONSOR. Tom Callaway offered sponsorship in bug 177235 (and bugzilla change-several-bugs-at-once feature requires me to add/edit something). -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 187609] Review Request: tre - POSIX compatible regexp library with approximate matching
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tre - POSIX compatible regexp library with approximate matching https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187609 [EMAIL PROTECTED] changed: What|Removed |Added QAContact|fedora-extras- |fedora-package- |[EMAIL PROTECTED] |[EMAIL PROTECTED] [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-05-13 22:12 EST --- Greetings. I plan on reviewing this package... before I do the full review however, it appears that upstream has released a new version (0.7.3) and changed the license to LGPL. Can you update to the new version and change the license tag in the spec? You may also consider (not a requirement or blocker) using a dist tag in your package, see: http://fedoraproject.org/wiki/DistTag I'll wait for the new version to do a full review. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
