search

astrand / xclip

Command line interface to the X11 clipboard
GNU General Public License v2.0
1.03k stars 73 forks source link

xclip-copyfile: mktemp -t option #108

Open nunotexbsd opened 3 years ago

nunotexbsd commented 3 years ago

Hello,

FreeBSD port needs a patch so xclip-copyfile works correctly:

-archive=mktemp || exit 1 +archive=mktemp -t xclip || exit 1

Could this be included upstream?

"If the -t prefix option is given, mktemp will generate a template string based on the prefix and the TMPDIR environment variable if set. The default location if TMPDIR is not set is /tmp. Care should be taken to ensure that it is appropriate to use an environment variable potentially supplied by the user."

Thanks,

Nuno Teixeira

nunotexbsd commented 3 years ago

I think this change will not affect other operating systems and it should improve portability.