[Bug 466737] Review Request: matio - Library for reading/writing Matlab MAT files

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





--- Comment #3 from Jason Tibbitts <tibbs@xxxxxxxxxxx>  2008-10-16 15:39:03 EDT ---
I'm not sure I understand about the patch fuzz; are you saying that gendiff
generates patches which need fuzz to apply or that the F8-generated patches
simply need fuzz to apply on rawhide?  Either of those cases seems somewhat
bizarre to me.

I understand that there are muliple ways to disable rpath; my concern is
thatthe  sed call shouldn't be there at all if it's commented out.  At least
not without some explanatory comment about it.

I understand that it doesn't seem interesting include both PDF and HTML
documentation; my point is that I can't tell from the comments in your spec
that you have code to accomplish that.  "#Fake the pdf creation" isn't
sufficient elaboration.

> The problem with the path for the fortran files is that they are listed in the
> src/Makefile.am whereas present in the src/fortran/ directory. 

Well, your comments indicate that the problem with this is improper debuginfo
generation.  Is that the case?

All I'm asking is that you comment your spec such that I can understand it. 
This spec has several different hacks in it, yet it doesn't describe why
they're needed clearly enough that someone else can look at it and understand
why the hacks are all there.

-- 
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]