[Bug 482884] Review Request: CLC-INTERCAL - Compiler Language With No Pronounceable Acronym

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


Jochen Schmitt <jochen@xxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jochen@xxxxxxxxxxxxxxx




--- Comment #1 from Jochen Schmitt <jochen@xxxxxxxxxxxxxxx>  2009-01-28 12:47:46 EDT ---
Question: It this your first Fedora package?

I ask this, because the package look like it's need some love.

I try to summarize my first mind about this package:

1. At first, you should try to get a better description the package, so a user
which want to install the package may got a idea, why he should try out this
new programming language

2. Your have specified 0 as the global package version and add version
information on the definition of subpackage. This is not ok, you should put the
version information on the Version statement and the subpackage shouldn't have
any version information in thier name.

3.) It's look very ugly, that you are using rpm macros for the whole content of
the Summary and Group statement.

Unfortunately, I either a sponsor or a perl packaging specialist, so I can't
review your package, but I would show your some points which are needs some
homework.

If this you first package and you need a sponsor please set the FE-NEEDSPONSOR
bug as a blocker.

Best Regards:

Jochen Schmitt

-- 
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@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]