Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xclip-copyfile: mktemp -t option #108

Open
nunotexbsd opened this issue Jan 9, 2021 · 1 comment
Open

xclip-copyfile: mktemp -t option #108

nunotexbsd opened this issue Jan 9, 2021 · 1 comment

Comments

@nunotexbsd
Copy link

nunotexbsd commented Jan 9, 2021

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
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant