[Bug 187932] Review Request: paraview - Parallel visualization application

[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 report.

Summary: Review Request: paraview - Parallel visualization application


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





------- Additional Comments From pertusus@xxxxxxx  2006-04-12 13:03 EST -------
Therre are still some rpmlint warning about executable files that shouldnt be
executable. I tracked down the issues.

One is with the install of files, you should add -m644 to the install call.

The other is with the find. Now the *.txt and *.xml aren't taken into accound
anymore...

This is fixed with:
find . \(-name \*.txt -o -name \*.xml -o -name \*.'[ch]*' \) -print0

Also \*.'[ch]*' finds a lot of files, including with *.cmake, .cvsignore,
*.ctest, *.h.in, *.check_cache, *.cg, *.html, *.cur. But it needs to find *.c
*.h *.hxx *.cxx *.cpp *.xml *.txt, so it may be right to catch more. Indeed
everything but scripts and directories should be chmod -x, so advise.

Please fix those and I believe it would be right.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

-- 
fedora-extras-list mailing list
fedora-extras-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-extras-list

[Index of Archives]     [Fedora General Discussion]     [Fedora Art]     [Fedora Docs]     [Fedora Package Review]     [Fedora Desktop]     [Big List of Linux Books]     [Yosemite Backpacking]     [KDE Users]

  Powered by Linux