Bug 148235
| Summary: | Supress noisy, always fail test condition in devel/xdg-utils | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Joseph S. Atkinson <jsa> | ||||
| Component: | Individual Port(s) | Assignee: | freebsd-gnome (Nobody) <gnome> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | ||||||
| Priority: | Normal | ||||||
| Version: | Latest | ||||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
Responsible Changed From-To: freebsd-bugs->gnome This is a port pr, assing to gnome. State Changed From-To: open->patched Will be committed after 8.1-R is out. State Changed From-To: patched->closed Committed thanks. kwm 2010-07-25 14:23:22 UTC
FreeBSD ports repository
Modified files:
devel/xdg-utils Makefile
Added files:
devel/xdg-utils/files patch-scripts_xdg-screensaver
Log:
mv on FreeBSD doesn't support the -T switch. Patch out the usage of it.
PR: ports/148235
Submitted by: "Joseph S. Atkinson" <jsa@wickedmachine.net>
Revision Changes Path
1.10 +2 -2 ports/devel/xdg-utils/Makefile
1.1 +16 -0 ports/devel/xdg-utils/files/patch-scripts_xdg-screensaver (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
|
xdg-screensaver performs a check for `mv -T` to safely move lockfiles to /tmp. FreeBSD does not support the -T switch, which causes mv to complain and spit out a usage example. If an app is run from the console (such as vlc), when calls to xdg-screensaver are made, it spews this error information to the terminal, possibly leading the user to believe the application is attempting to misappropriately move their data. Fix: This patch (which should just be inserted into ${FILESDIR}) comments out the offending test condition and replaces it with the `false` command to essentially skip to the else condition without noisy error messages. IMPORTANT: You must remove the .txt extension required by web based pr interface. Patch attached with submission follows: How-To-Repeat: Run multimedia/vlc from a terminal, opening any video multimedia type. Watch for scary message that the app failed to `mv` something...