Bug 249075 - x11/xclip: fix xclip-pastefile
Summary: x11/xclip: fix xclip-pastefile
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Dmitri Goutnik
Depends on:
Reported: 2020-09-03 09:43 UTC by Tomohiro Hosaka
Modified: 2020-09-19 09:16 UTC (History)
2 users (show)

See Also:

patch (1.26 KB, patch)
2020-09-19 09:16 UTC, Tomohiro Hosaka
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tomohiro Hosaka 2020-09-03 09:43:57 UTC
tar is used in the xclip-pastefile.
I get the following error:

    # xclip-pastefile
    tar: Error opening archive: Failed to open '/dev/sa0'

    It will be fixed below.
    --- xclip-pastefile.orig	2016-09-13 07:09:12 UTC
    +++ xclip-pastefile
    @@ -4,4 +4,4 @@ if [ "x$1" != "x" ]; then
        echo "Usage: $0" >&2
        exit 1
    -xclip -selection secondary -o | gunzip -c | tar xv
    +xclip -selection secondary -o | gunzip -c | tar xv -f -

According to the Makefile style, is the below suitable?

    --- Makefile.orig	2020-09-03 18:40:55.121010000 +0900
    +++ Makefile	2020-09-03 18:35:11.439860000 +0900
    @@ -27,6 +27,7 @@

            ${REINPLACE_CMD} 's,mktemp,mktemp -t xclip,' ${WRKSRC}/xclip-copyfile
    +	${REINPLACE_CMD} 's,tar xv,tar xv -f -,' ${WRKSRC}/xclip-pastefile

            ${STRIP_CMD} ${STAGEDIR}${PREFIX}/bin/xclip

Comment 1 Dmitri Goutnik freebsd_committer 2020-09-03 21:01:46 UTC
(In reply to Tomohiro Hosaka from comment #0)

Thanks for the patch. Looks like this was already fixed upstream [1] but the fix didn't make it into xclip-0.13. Would you mind updating PR to pull the patch from Github?

Also, since this port is currently unmaintained, how do you feel about taking over the maintainership?

[1] https://github.com/astrand/xclip/commit/db74a39e15f1a75bb1f828103ae7a8d2b4d8ccc3
Comment 2 Tomohiro Hosaka 2020-09-19 09:16:59 UTC
Created attachment 218069 [details]


Thanks for your advice.

I followed the advice.

There is no problem changing the maintainer.