Skip to content

Add Xeyes 1.1.1 #459

Merged
merged 1 commit into from
Aug 18, 2017
Merged

Add Xeyes 1.1.1 #459

merged 1 commit into from
Aug 18, 2017

Conversation

thomas
Copy link
Collaborator

@thomas thomas commented Aug 16, 2017

xeyes helps you to find your mouse on the desktop. Also it's somehow a
real classical application on X.

xeyes helps you to find your mouse on the desktop. Also it's somehow a
real classical application on X.
@thomas
Copy link
Collaborator Author

thomas commented Aug 16, 2017

Note, this one was also a test for 'root-install tests' which don't interfere with real root installs.
Thus I've choosen a completely harmless one in case of interference :)

@pmenzel
Copy link
Collaborator

pmenzel commented Aug 16, 2017

I am unable to install it.

$ sudo bee install xeyes
bee-install: xeyes: No packages available matching your query.

What is “root-install tests”?

@thomas
Copy link
Collaborator Author

thomas commented Aug 16, 2017

Yes right, and I forgot to mention that there should be no traces of xeyes on deinemuddah.
So a 'locate xeyes' tomorrow morning should report only one hit (/usr/share/icons/mono/scalable/apps/xeyes.svgz).
The rationale behind this is the handling of development installations (including bee files) on machines that aren't disted.

@thomas
Copy link
Collaborator Author

thomas commented Aug 17, 2017

The test succeed, and I've just built it in the ordinary way. Now erverybody might be able to test this magnificent tool.

@donald
Copy link
Collaborator

donald commented Aug 18, 2017

I don't understand “root-install tests” or "development installations (including bee files) on machines that aren't disted" . However, the packages looks okay. So, you thinks its useful to find you mouse and should be disted? I have no objection. Go ahead and merge yourself. I won't because I'm not 100% sure you're suggesting this or whether you just wanted to test some private bee tool. On the other hand, if that was the case, there would be no reason to create a pull request....

@thomas
Copy link
Collaborator Author

thomas commented Aug 18, 2017

Well, yes, it was a test of a bee tool and also I wanted to have the xeyes disted. Maybe it would have been better to tell more clearly to ignore this request for 24 hours, or to test it silently and than proceed as usual.

@pmenzel pmenzel changed the title xeyes: Add version 1.1.1 Add Xeyes 1.1.1 Aug 18, 2017
@pmenzel pmenzel merged commit 02835ac into master Aug 18, 2017
@donald
Copy link
Collaborator

donald commented Aug 18, 2017

If for some reason you want a pull request which is not (or not yet) ready to be merged, (e.g. is just created to invite discussion), you can add the "not ready for merge" label.

@thomas
Copy link
Collaborator Author

thomas commented Aug 18, 2017

will do, I've never used the tag thing before.

@donald donald deleted the add-xeyes-1.1.1 branch October 9, 2017 11:13
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants