[Bug 529496] Review Request: libmtag - An advanced C music tagging library with a simple API

[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=529496

--- Comment #48 from Ralf Corsepius <rc040203@xxxxxxxxxx> 2010-01-13 13:17:05 EST ---
(In reply to comment #36)
> (In reply to comment #34)
> > Package still does not build:
> 
> [...]
> 
> > Apparently you don't test the stuff you want us to review.
> 
> Why do you have to assume the worst?
As I wrote before: The way this review proceeds speaks for itself.

IMO, it's only thanks to Michael's seemingly endless patience, this review has
not yet found a premature end.

> Maybe you should try to think of another
> non-antagonist option (hint: 64).
Pardon? It's a reviewer's job to test your work and to point out your bugs.

If you didn't test on 64bit platforms, this is your fault.

> > Well, you are not the first person who is outsmarting himself by using
> > handwritten makefiles.
> 
> That's an uncalled for assumption, and ad hominem attack.
No, it's 20 years of SW development experience.

As I tried to tell you before, you are not the first persoon who is switching
to manually written makefiles without being aware about the bugs and short
comings his approach carries - There have been generations of programmers
before you.

> What do you even know about me?    
Well, to me, it's apparent that you are at the very beginning of getting
started with packaging and system integration.

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review

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