[Bug 447844] Review Request: krazy2 - KDE code checking tool

2008-09-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=447844


Ben Boeckel [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #21 from Ben Boeckel [EMAIL PROTECTED]  2008-09-23 08:39:44 EDT 
---
Just did it this morning (was away for the weekend). Closing.

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-20 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=447844





--- Comment #19 from Kevin Fenzi [EMAIL PROTECTED]  2008-09-20 20:32:19 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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-19 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=447844





--- Comment #15 from Ben Boeckel [EMAIL PROTECTED]  2008-09-19 08:26:46 EDT 
---
(In reply to comment #14)
 * One issue
   - Requires: desktop-file-utils is not needed.
Actually, it is. Krazy uses desktop-file-validate (it has 0.14 upstream, 0.15
is in Fedora, so I went with that). Should I comment that in the spec file?

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-19 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=447844





--- Comment #16 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-19 09:06:25 EDT 
---
(In reply to comment #15)
 (In reply to comment #14)
  * One issue
- Requires: desktop-file-utils is not needed.
 Actually, it is. Krazy uses desktop-file-validate (it has 0.14 upstream, 0.15
 is in Fedora, so I went with that). Should I comment that in the spec file?

Ah, okay. Then I recommend to write in a spec file.

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-19 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=447844


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #17 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-19 09:47:26 EDT 
---
(Now I am sponsoring the submitter)

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-19 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=447844





--- Comment #18 from Ben Boeckel [EMAIL PROTECTED]  2008-09-19 10:21:27 EDT 
---
New Package CVS Request
===
Package Name: krazy2
Short Description: KDE code checking tool
Owners: mathstuf
Branches: F-8 F-9
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-18 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=447844





--- Comment #10 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-18 02:08:04 EDT 
---
Still rebuild fails: needs investigating...
http://koji.fedoraproject.org/koji/taskinfo?taskID=831427

By the way, are you trying rpmbuild as root? If so, please try
rpmbuild as a normal user. After you setup rpmdev tree by
rpmdev-setuptree (in rpmdevtools rpm) as a normal user, you should be able to 
try rpmbuild as the user.

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-18 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=447844





--- Comment #11 from Ben Boeckel [EMAIL PROTECTED]  2008-09-18 11:13:49 EDT 
---
Yes, I run (ran) it as root. Fixed it up. Thanks.

http://benboeckel.net/krazy2/krazy2.spec
http://benboeckel.net/krazy2/krazy2-2.6-3.fc9.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.

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


[Bug 447844] Review Request: krazy2 - KDE code checking tool

2008-09-18 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=447844


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #12 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-18 13:19:58 EDT 
---
For 2.6-3:
* Source0
  - Source tarball must be written by full URL
https://fedoraproject.org/wiki/Packaging/SourceURL

* BuildRequires
---
   327  + pushd doc
   328  + make
DESTDIR=/builddir/build/BUILDROOT/krazy2-2.6-3.fc10.i386/usr/share/man install
   329  perldoc krazyrc.pod  krazyrc.5
!!!330  sh: nroff: command not found
   331  list=krazyrc.5; for man in $list; do \
   332  echo Installing man page $man; \
   333  mv -f $man
/builddir/build/BUILDROOT/krazy2-2.6-3.fc10.i386/usr/share/man/man5/$man; \
   334  done
   335  Installing man page krazyrc.5
   336  + popd
---
   - Perhaps groff is needed for BuildRequires.

* man files
  - Files under %_mandir are automatically regarded as %doc.

* Directory ownership issue
  - The directory %{perl_vendorlib}/Krazy/ is not owned by any packages

* Documents
  - Please add the following files to %doc.
---
COPYING
README
---

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-18 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=447844





--- Comment #13 from Ben Boeckel [EMAIL PROTECTED]  2008-09-18 13:53:34 EDT 
---
Fixed. Also added the svn revision number as upstream doesn't bump the version
when it gets updated.

http://benboeckel.net/krazy2/krazy2.spec
http://benboeckel.net/krazy2/krazy2-2.6-4.20080918svn862357.fc9.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.

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


[Bug 447844] Review Request: krazy2 - KDE code checking tool

2008-09-18 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=447844





--- Comment #14 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-19 01:50:55 EDT 
---
Okay.

* One issue
  - Requires: desktop-file-utils is not needed.

--
 This package (krazy2) is APPROVED by mtasaka
--

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-17 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=447844





--- Comment #8 from Ben Boeckel [EMAIL PROTECTED]  2008-09-17 20:34:52 EDT ---
Got them more up-to-date here:
http://benboeckel.net/krazy2/krazy2.spec
http://benboeckel.net/krazy2/krazy2-2.6-1.fc9.src.rpm

Here are some of the questions that I have that need attention:
1) There is one file that needs compiling (src/passbyvalue/passbyvalue.cpp)
while everything else is noarch (perl scripts)

2) rpmlint errors saying that there are no binaries and it should be noarch,
but that plugin is not noarch as it needs compiled. rpmlint doesn't find it and
errors saying that it should then be in /usr/share.

Any ideas as to how I should handle that? Maybe an arch-dependent subpackage
from a noarch base package?

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-17 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=447844





--- Comment #9 from Ben Boeckel [EMAIL PROTECTED]  2008-09-17 20:50:54 EDT ---
False alarm, thought I had fixed a bug before and didn't think it came from
there:
http://benboeckel.net/krazy2/krazy2.spec
http://benboeckel.net/krazy2/krazy2-2.6-2.fc9.src.rpm

Fixed it. Had to patch to get it to accept LIBSUFFIX in passbyvalue. Now it
actually installs to the right place.

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-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=447844





--- Comment #5 from Ben Boeckel [EMAIL PROTECTED]  2008-09-17 00:45:52 EDT ---
I no longer need sponsored.
See: https://bugzilla.redhat.com/show_bug.cgi?id=450371

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-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=447844


Ben Boeckel [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks|177841  |




-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-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=447844


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||177841




--- Comment #6 from Mamoru Tasaka [EMAIL PROTECTED]  2008-09-17 01:36:47 EDT 
---
(Re-adding NEEDSPONSOR: see my comment on bug 450371)

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-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=447844


Kevin Kofler [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #7 from Kevin Kofler [EMAIL PROTECTED]  2008-09-17 01:46:45 EDT 
---
Of course this fails, the make PREFIX=/usr install for the plugins doesn't
use RPM_BUILD_ROOT anywhere, so they aren't installed to the right place.

-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-09-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=447844


Itamar Reis Peixoto [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
  Alias||krazy2




-- 
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 447844] Review Request: krazy2 - KDE code checking tool

2008-07-31 Thread bugzilla
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: krazy2 - KDE code checking tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-31 10:05 EST ---
Rebuild failed:
http://koji.fedoraproject.org/koji/taskinfo?taskID=750060

-- 
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, or are watching someone who is.

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


[Bug 447844] Review Request: krazy2 - KDE code checking tool

2008-06-16 Thread bugzilla
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: krazy2 - KDE code checking tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-16 20:14 EST ---
The spec and SRPM have been relocated:
http://www.hypersonicosft.org/temp/krazy2-1.11-1.fc9.src.rpm
http://www.hypersonicsoft.org/temp/krazy2.spec

-- 
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, or are watching someone who is.

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


[Bug 447844] Review Request: krazy2 - KDE code checking tool

2008-05-21 Thread bugzilla
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: krazy2 - KDE code checking tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-22 00:09 EST ---
Forgot to mention that this is my first package and that I need sponsored.

-- 
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, or are watching someone who is.

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


[Bug 447844] Review Request: krazy2 - KDE code checking tool

2008-05-21 Thread bugzilla
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: krazy2 - KDE code checking tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




-- 
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, or are watching someone who is.

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


[Bug 447844] Review Request: krazy2 - KDE code checking tool

2008-05-21 Thread bugzilla
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: krazy2 - KDE code checking tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-22 01:39 EST ---
After thinking about it a bit, should the plugins in the package be a separate
RPM (krazy2-plugins perhaps)?

-- 
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, or are watching someone who is.

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