[Patch][jethro][master] xcursor-transparent-theme: insert several cursors

[Patch][jethro][master] xcursor-transparent-theme: insert several cursors

[Johannes Pointner]

This patch inserts several cursors which are used by Qt for example.

Signed-off-by: Johannes Pointner <johannes.pointner@gmail.com>
---
 .../insert_several_cursors.patch                   | 79 ++++++++++++++++++++++
 .../xcursor-transparent-theme_0.1.1.bb             |  3 +-
 2 files changed, 81 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme-0.1.1/insert_several_cursors.patch

diff --git a/meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme-0.1.1/insert_several_cursors.patch b/meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme-0.1.1/insert_several_cursors.patch
new file mode 100644
index 0000000..861899e
--- /dev/null
+++ b/meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme-0.1.1/insert_several_cursors.patch
@@ -0,0 +1,79 @@
+--- xcursor-transparent-theme-0.1.1.orig/cursors/Makefile.am	2016-03-01 21:48:25.000000000 +0100
++++ xcursor-transparent-theme-0.1.1/cursors/Makefile.am	2016-03-01 22:10:06.473326873 +0100
+@@ -26,10 +26,17 @@
+ 	bottom_tee \
+ 	center_ptr \
+ 	circle \
++	closedhand \
++	color-picker \
++	copy \
+ 	cross \
+ 	cross_reverse \
+ 	crossed_circle \
+ 	crosshair \
++	dnd-copy \
++	dnd-move \
++	dnd-no-drop \
++	dnd-none \
+ 	dot \
+ 	dot_box_mask \
+ 	dotbox \
+@@ -40,21 +47,27 @@
+ 	exchange     \
+ 	fd_double_arrow \
+ 	fleur \
++	forbidden \
+ 	gumby \
+ 	h_double_arrow \
+ 	hand \
+ 	hand1 \
+ 	hand2 \
++	ibeam \
+ 	left_ptr \
+ 	left_ptr_watch \
+ 	left_side \
+ 	left_tee  \
++	link \
+ 	ll_angle  \
+ 	lr_angle  \
+ 	move      \
++	open_hand \
+ 	pencil    \
+ 	pirate    \
+ 	plus      \
++	pointing_hand \
++	progress \
+ 	question_arrow \
+ 	right_ptr      \
+ 	right_side     \
+@@ -67,9 +80,17 @@
+ 	sb_up_arrow \
+ 	sb_v_double_arrow \
+ 	shuttle \
++	size_all \
++	size_bdiag \
++	size_fdiag \
++	size_hor \
++	size_ver \
+ 	sizing  \
++	split_h \
++	split_v \
+ 	target  \
+ 	tcross  \
++	text \
+ 	top_left_arrow  \
+ 	top_left_corner \
+ 	top_right_corner \
+@@ -77,9 +98,12 @@
+ 	top_tee \
+ 	trek \
+ 	ul_angle \
++	up_arrow \
+ 	ur_angle \
+ 	v_double_arrow \
++	wait \
+ 	watch \
++	whats_this \
+ 	xterm  
+ 
+ CURSOR_DIR = $(datadir)/icons/xcursor-transparent/cursors
diff --git a/meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme_0.1.1.bb b/meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme_0.1.1.bb
index 5e668ab..5a9df62 100644
--- a/meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme_0.1.1.bb
+++ b/meta/recipes-graphics/xcursor-transparent-theme/xcursor-transparent-theme_0.1.1.bb
@@ -10,7 +10,8 @@ PR = "r4"
 
 SRC_URI = "http://downloads.yoctoproject.org/releases/matchbox/utils/xcursor-transparent-theme-${PV}.tar.gz \
 	   file://use-relative-symlinks.patch \
-	   file://fix_watch_cursor.patch"
+	   file://fix_watch_cursor.patch \
+	   file://insert_several_cursors.patch"
 
 SRC_URI[md5sum] = "7b0c623049d4aab20600d6473f8aab23"
 SRC_URI[sha256sum] = "b26adf2d503d01299718390ae39dab4691a67220de09423be0364e9a060bf7e4"
-- 
2.7.2

Re: [Patch][jethro][master] xcursor-transparent-theme: insert several cursors

[Richard Purdie]

On Tue, 2016-03-01 at 22:39 +0100, Johannes Pointner wrote:
> This patch inserts several cursors which are used by Qt for example.
> 
> Signed-off-by: Johannes Pointner <johannes.pointner@gmail.com>
> ---
>  .../insert_several_cursors.patch                   | 79
> ++++++++++++++++++++++
>  .../xcursor-transparent-theme_0.1.1.bb             |  3 +-
>  2 files changed, 81 insertions(+), 1 deletion(-)
>  create mode 100644 meta/recipes-graphics/xcursor-transparent
> -theme/xcursor-transparent-theme-0.1.1/insert_several_cursors.patch
> 
> diff --git a/meta/recipes-graphics/xcursor-transparent-theme/xcursor
> -transparent-theme-0.1.1/insert_several_cursors.patch b/meta/recipes
> -graphics/xcursor-transparent-theme/xcursor-transparent-theme
> -0.1.1/insert_several_cursors.patch
> new file mode 100644
> index 0000000..861899e
> --- /dev/null
> +++ b/meta/recipes-graphics/xcursor-transparent-theme/xcursor
> -transparent-theme-0.1.1/insert_several_cursors.patch
> @@ -0,0 +1,79 @@

There is no description of the patch here, what it does, who wrote it,
what the upstream status is etc. Please read the patch submission
guidelines.

How have we survived so long without needing this if its so important?

Cheers,

Richard

Re: [Patch][jethro][master] xcursor-transparent-theme: insert several cursors

[Johannes Pointner]

Hello Richard,

sorry I forgot to add a description into the patch.
The upstream status is quite difficult to define because I don't know
if there is an upstream for this package.

I'm using Qt5 with X11 and have seen some cursor even if the
transparent-theme is set because of the missing cursors. But maybe
this is just something I should add by a bbappend file.
I thought it could be useful to other people which are using a similar setup.

Thanks,
Hannes


2016-03-01 23:07 GMT+01:00 Richard Purdie <richard.purdie@linuxfoundation.org>:
> On Tue, 2016-03-01 at 22:39 +0100, Johannes Pointner wrote:
>> This patch inserts several cursors which are used by Qt for example.
>>
>> Signed-off-by: Johannes Pointner <johannes.pointner@gmail.com>
>> ---
>>  .../insert_several_cursors.patch                   | 79
>> ++++++++++++++++++++++
>>  .../xcursor-transparent-theme_0.1.1.bb             |  3 +-
>>  2 files changed, 81 insertions(+), 1 deletion(-)
>>  create mode 100644 meta/recipes-graphics/xcursor-transparent
>> -theme/xcursor-transparent-theme-0.1.1/insert_several_cursors.patch
>>
>> diff --git a/meta/recipes-graphics/xcursor-transparent-theme/xcursor
>> -transparent-theme-0.1.1/insert_several_cursors.patch b/meta/recipes
>> -graphics/xcursor-transparent-theme/xcursor-transparent-theme
>> -0.1.1/insert_several_cursors.patch
>> new file mode 100644
>> index 0000000..861899e
>> --- /dev/null
>> +++ b/meta/recipes-graphics/xcursor-transparent-theme/xcursor
>> -transparent-theme-0.1.1/insert_several_cursors.patch
>> @@ -0,0 +1,79 @@
>
> There is no description of the patch here, what it does, who wrote it,
> what the upstream status is etc. Please read the patch submission
> guidelines.
>
> How have we survived so long without needing this if its so important?
>
> Cheers,
>
> Richard
>
>

Re: [Patch][jethro][master] xcursor-transparent-theme: insert several cursors

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 382 bytes --]

On 1 March 2016 at 22:22, Johannes Pointner <h4nn35.work@gmail.com> wrote:

> The upstream status is quite difficult to define because I don't know
> if there is an upstream for this package.
>

We're upstream:
http://git.yoctoproject.org/cgit/cgit.cgi/xcursor-transparent-theme/.
Please submit this as a patch to that layer and then a srcrev bump for the
recipe.

Ross

[-- Attachment #2: Type: text/html, Size: 941 bytes --]