[Bug 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-12 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=461139


Nicolas Mailhot [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||needinfo?([EMAIL PROTECTED]
   ||.com)




--- Comment #16 from Nicolas Mailhot [EMAIL PROTECTED]  2008-10-12 12:44:54 
EDT ---
(In reply to comment #14)
 ok ..will fix them soon

I'll review it just as soon as you post a new spec file, I promise :)

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-12 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=461139





--- Comment #17 from Jens Petersen [EMAIL PROTECTED]  2008-10-12 20:40:58 EDT 
---
(In reply to comment #15)
  Nicolas, I feel like this would be a good candidate for naming
  arabeyes-Thabit-fonts.
 
 lowercase please

The upstream name is clearly Thabit: do you think we should lowercase it?
Personally I usually prefer to preserve upstream casing as far as possible.

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-05 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=461139





--- Comment #14 from Subhodip Biswas [EMAIL PROTECTED]  2008-10-05 14:27:03 
EDT ---
ok ..will fix them soon

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-05 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=461139


Nicolas Mailhot [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #15 from Nicolas Mailhot [EMAIL PROTECTED]  2008-10-05 16:33:54 
EDT ---
lowercase(In reply to comment #9)
 Nicolas, I feel like this would be a good candidate for naming
 arabeyes-Thabit-fonts.

lowercase please

Appart from that, I really need to find a few hours to review the font packages
backlog :(

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-03 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=461139





--- Comment #11 from Jens Petersen [EMAIL PROTECTED]  2008-10-03 02:14:59 EDT 
---
Created an attachment (id=319318)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=319318)
Thabit-fonts.spec-1.patch

Bit more cleanup now that Mothanna-fonts is going to be a separate 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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-03 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=461139


Jens Petersen [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||462711




--- Comment #10 from Jens Petersen [EMAIL PROTECTED]  2008-10-03 02:10:59 EDT 
---
Strictly this doesn't block bug 462711 (Mothanna-fonts review) but in practice
probably most of the changes to this packaging will need to be done for the
other package too, so making it block.

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-03 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=461139


Jens Petersen [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks|462711  |




--- Comment #12 from Jens Petersen [EMAIL PROTECTED]  2008-10-03 02:19:24 EDT 
---
rpmlint says:

Thabit-fonts.noarch: W: wrong-file-end-of-line-encoding
/usr/share/doc/Thabit-fonts-0.02/OFL.txt
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-03 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=461139





--- Comment #13 from Jens Petersen [EMAIL PROTECTED]  2008-10-03 02:21:54 EDT 
---
Well comment 12 is after fixing rpmlint warning for ver-rel tagging syntax in
the changelog:

 %changelog
-* Mon Sep 08 2008  Subhodip Biswas [EMAIL PROTECTED] -0.02-3
-- Packaged seperatly as  Thabit and Mothana. 
-* Tue Sep 02 2008  Subhodip Biswas [EMAIL PROTECTED] -0.02-2
+* Mon Sep 08 2008  Subhodip Biswas [EMAIL PROTECTED] - 0.02-3
+- Packaged seperately as Thabit and Mothana. 
+
+* Tue Sep 02 2008  Subhodip Biswas [EMAIL PROTECTED] - 0.02-2
 - Submit for review. 
+
 * Fri Aug 29 2008  Khaled Hosny [EMAIL PROTECTED] - 0.02-1

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-03 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=461139


Jens Petersen [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||462711




-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

2008-10-02 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=461139





--- Comment #9 from Jens Petersen [EMAIL PROTECTED]  2008-10-03 00:05:51 EDT 
---
Nicolas, I feel like this would be a good candidate for naming
arabeyes-Thabit-fonts.
Maybe after f10 we should consider revising the fonts packaging naming
guidelines?

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

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





--- Comment #8 from Subhodip Biswas [EMAIL PROTECTED]  2008-09-18 10:09:27 
EDT ---
big mistake : i accidently build the font with tag dist-f9 in koji .however
this is  corrected .

new koji build : http://koji.fedoraproject.org/koji/taskinfo?taskID=831897

-- 
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 461139] Review Request: Thabit-fonts -Thabit-fonts from Arabeyes.org

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


Subhodip Biswas [EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |arabeyes-core-fonts - Core  |Thabit-fonts -Thabit-fonts
   |Arabic fonts from   |from Arabeyes.org
   |Arabeyes.org|




--- Comment #7 from Subhodip Biswas [EMAIL PROTECTED]  2008-09-14 11:28:45 
EDT ---
koji build for F9 : http://koji.fedoraproject.org/koji/taskinfo?taskID=825141
New spec : http://subhodip.fedorapeople.org/Thabit-fonts.spec
 New SRPM : http://subhodip.fedorapeople.org/Thabit-fonts-0.02-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