aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCy Schubert <cy@FreeBSD.org>2023-10-16 05:34:04 +0000
committerCy Schubert <cy@FreeBSD.org>2023-12-05 22:39:03 +0000
commitfeec232fadd3d30c62282b6dd218536b7322e761 (patch)
treedd5bb4b90aaf33b6f76c7e5e12c212cdf2fd6e79
parent31ffe63e69db1e53cd094debaedf286f4290e05e (diff)
downloadports-feec232fadd3d30c62282b6dd218536b7322e761.tar.gz
ports-feec232fadd3d30c62282b6dd218536b7322e761.zip
x11-toolkits/open-motif*: EditRes functionality is unreliable on 64 bit
Like x11-toolkits/libXmu, the same problem exists in Motif and editres(1) will not pop up the resource box, displaying, "this widget no longer exists in the client." Additional discussion and patch posted near the end of the discussion at, https://gitlab.freedesktop.org/xorg/lib/libxmu/-/issues/1 MFH: 2023Q4
-rw-r--r--x11-toolkits/open-motif-devel/Makefile2
-rw-r--r--x11-toolkits/open-motif-devel/files/patch-lib_Xm_EditresCom.c127
-rw-r--r--x11-toolkits/open-motif/Makefile2
-rw-r--r--x11-toolkits/open-motif/files/patch-lib_Xm_EditresCom.c127
4 files changed, 256 insertions, 2 deletions
diff --git a/x11-toolkits/open-motif-devel/Makefile b/x11-toolkits/open-motif-devel/Makefile
index ec74a5e473a9..d64c1dbe67d9 100644
--- a/x11-toolkits/open-motif-devel/Makefile
+++ b/x11-toolkits/open-motif-devel/Makefile
@@ -1,6 +1,6 @@
PORTNAME= open-motif
PORTVERSION= ${COMMIT_DATE}
-PORTREVISION= 6
+PORTREVISION= 7
CATEGORIES= x11-toolkits
PKGNAMESUFFIX= -devel
diff --git a/x11-toolkits/open-motif-devel/files/patch-lib_Xm_EditresCom.c b/x11-toolkits/open-motif-devel/files/patch-lib_Xm_EditresCom.c
new file mode 100644
index 000000000000..6eff286772da
--- /dev/null
+++ b/x11-toolkits/open-motif-devel/files/patch-lib_Xm_EditresCom.c
@@ -0,0 +1,127 @@
+--- lib/Xm/EditresCom.c.orig 2023-02-15 16:42:34.000000000 -0800
++++ lib/Xm/EditresCom.c 2023-10-16 20:14:52.602297000 -0700
+@@ -183,6 +183,39 @@
+ }
+ #endif
+
++#define HAVE_XMU_GET_WIDGET_INFO_RESOLVE 1
++#ifdef HAVE_XMU_GET_WIDGET_INFO_RESOLVE
++static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ return _XEditResGetWidgetInfoResolve(stream, info);
++}
++#else
++static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ return _XEditResGetWidgetInfo(stream, info);
++}
++#endif
++
++#define HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE 1
++#ifdef HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE
++static void PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ _XEditResPutWidgetInfoUnresolve(stream, info);
++}
++#else
++static Boolean PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ _XEditResPutWidgetInfo(stream, info);
++}
++#endif
++
++#define HAVE_XMU_PUT_WIDGET_UNRESOLVE 1
++#ifdef HAVE_XMU_PUT_WIDGET_UNRESOLVE
++static void PutWidget(ProtocolStream *stream, Widget widget) {
++ _XEditResPutWidgetUnresolve(stream, widget);
++}
++#else
++static void PutWidget(ProtocolStream *stream, Widget widget) {
++ _XEditResPut32(stream, widget);
++}
++#endif
++
+ /************************************************************
+ *
+ * Resource Editor Communication Code
+@@ -352,7 +385,7 @@
+ XtCalloc(sizeof(WidgetInfo), sv_event->num_entries);
+
+ for (i = 0; i < sv_event->num_entries; i++) {
+- if (!_XEditResGetWidgetInfo(stream, sv_event->widgets + i))
++ if (!GetWidgetInfo(stream, sv_event->widgets + i))
+ goto done;
+ }
+ }
+@@ -364,7 +397,7 @@
+ find_event->widgets = (WidgetInfo *)
+ XtCalloc(sizeof(WidgetInfo), 1);
+
+- if (!(_XEditResGetWidgetInfo(stream, find_event->widgets) &&
++ if (!(GetWidgetInfo(stream, find_event->widgets) &&
+ _XEditResGetSigned16(stream, &(find_event->x)) &&
+ _XEditResGetSigned16(stream, &(find_event->y))))
+ {
+@@ -384,7 +417,7 @@
+ get_event->widgets = (WidgetInfo *)
+ XtCalloc(sizeof(WidgetInfo), get_event->num_entries);
+ for (i = 0; i < get_event->num_entries; i++) {
+- if (!_XEditResGetWidgetInfo(stream, get_event->widgets + i))
++ if (!GetWidgetInfo(stream, get_event->widgets + i))
+ goto done;
+ }
+ }
+@@ -397,7 +430,7 @@
+ _XEditResGet16(stream, &(gv_event->num_entries));
+ gv_event->widgets = (WidgetInfo *)
+ XtCalloc(sizeof(WidgetInfo), gv_event->num_entries);
+- _XEditResGetWidgetInfo(stream, gv_event->widgets);
++ GetWidgetInfo(stream, gv_event->widgets);
+ }
+ break;
+
+@@ -846,7 +879,7 @@
+
+ for (i = 0 ; i < sv_event->num_entries; i++) {
+ if ((str = VerifyWidget(w, &(sv_event->widgets[i]))) != NULL) {
+- _XEditResPutWidgetInfo(stream, &(sv_event->widgets[i]));
++ PutWidgetInfo(stream, &(sv_event->widgets[i]));
+ _XEditResPutString8(stream, str);
+ XtFree(str);
+ count++;
+@@ -973,7 +1006,7 @@
+ */
+
+ (*(info->count))++;
+- _XEditResPutWidgetInfo(info->stream, info->entry);
++ PutWidgetInfo(info->stream, info->entry);
+ _XEditResPutString8(info->stream, pbuf);
+ if (pbuf != buf)
+ XtFree(pbuf);
+@@ -1174,7 +1207,7 @@
+ * Send out the widget id.
+ */
+
+- _XEditResPutWidgetInfo(stream, &(geom_event->widgets[i]));
++ PutWidgetInfo(stream, &(geom_event->widgets[i]));
+ if ((str = VerifyWidget(w, &(geom_event->widgets[i]))) != NULL) {
+ _XEditResPutBool(stream, True); /* an error occured. */
+ _XEditResPutString8(stream, str); /* set message. */
+@@ -1420,7 +1453,7 @@
+ /*
+ * Send out the widget id.
+ */
+- _XEditResPutWidgetInfo(stream, &(res_event->widgets[i]));
++ PutWidgetInfo(stream, &(res_event->widgets[i]));
+ if ((str = VerifyWidget(w, &(res_event->widgets[i]))) != NULL) {
+ _XEditResPutBool(stream, True); /* an error occured. */
+ _XEditResPutString8(stream, str); /* set message. */
+@@ -1766,10 +1799,11 @@
+
+ _XEditResPut16(stream, num_widgets); /* insert number of widgets. */
+ for (i = 0; i < num_widgets; i++) /* insert Widgets themselves. */
+- _XEditResPut32(stream, widget_list[i]);
++ PutWidget(stream, (Widget)widget_list[i]);
+
+ XtFree((char *)widget_list);
+ }
++
+
+ #if 0
+ /************************************************************
diff --git a/x11-toolkits/open-motif/Makefile b/x11-toolkits/open-motif/Makefile
index 2723ca4a7381..91f5f39f22c9 100644
--- a/x11-toolkits/open-motif/Makefile
+++ b/x11-toolkits/open-motif/Makefile
@@ -1,6 +1,6 @@
PORTNAME= open-motif
PORTVERSION= 2.3.8
-PORTREVISION= 7
+PORTREVISION= 8
CATEGORIES= x11-toolkits
# MASTER_SITES= SF/${PORTNAME}/Motif%202.3.4%20Source%20Code/
MASTER_SITES= SF/motif/Motif%20${PORTVERSION}%20Source%20Code/
diff --git a/x11-toolkits/open-motif/files/patch-lib_Xm_EditresCom.c b/x11-toolkits/open-motif/files/patch-lib_Xm_EditresCom.c
new file mode 100644
index 000000000000..6eff286772da
--- /dev/null
+++ b/x11-toolkits/open-motif/files/patch-lib_Xm_EditresCom.c
@@ -0,0 +1,127 @@
+--- lib/Xm/EditresCom.c.orig 2023-02-15 16:42:34.000000000 -0800
++++ lib/Xm/EditresCom.c 2023-10-16 20:14:52.602297000 -0700
+@@ -183,6 +183,39 @@
+ }
+ #endif
+
++#define HAVE_XMU_GET_WIDGET_INFO_RESOLVE 1
++#ifdef HAVE_XMU_GET_WIDGET_INFO_RESOLVE
++static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ return _XEditResGetWidgetInfoResolve(stream, info);
++}
++#else
++static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ return _XEditResGetWidgetInfo(stream, info);
++}
++#endif
++
++#define HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE 1
++#ifdef HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE
++static void PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ _XEditResPutWidgetInfoUnresolve(stream, info);
++}
++#else
++static Boolean PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
++ _XEditResPutWidgetInfo(stream, info);
++}
++#endif
++
++#define HAVE_XMU_PUT_WIDGET_UNRESOLVE 1
++#ifdef HAVE_XMU_PUT_WIDGET_UNRESOLVE
++static void PutWidget(ProtocolStream *stream, Widget widget) {
++ _XEditResPutWidgetUnresolve(stream, widget);
++}
++#else
++static void PutWidget(ProtocolStream *stream, Widget widget) {
++ _XEditResPut32(stream, widget);
++}
++#endif
++
+ /************************************************************
+ *
+ * Resource Editor Communication Code
+@@ -352,7 +385,7 @@
+ XtCalloc(sizeof(WidgetInfo), sv_event->num_entries);
+
+ for (i = 0; i < sv_event->num_entries; i++) {
+- if (!_XEditResGetWidgetInfo(stream, sv_event->widgets + i))
++ if (!GetWidgetInfo(stream, sv_event->widgets + i))
+ goto done;
+ }
+ }
+@@ -364,7 +397,7 @@
+ find_event->widgets = (WidgetInfo *)
+ XtCalloc(sizeof(WidgetInfo), 1);
+
+- if (!(_XEditResGetWidgetInfo(stream, find_event->widgets) &&
++ if (!(GetWidgetInfo(stream, find_event->widgets) &&
+ _XEditResGetSigned16(stream, &(find_event->x)) &&
+ _XEditResGetSigned16(stream, &(find_event->y))))
+ {
+@@ -384,7 +417,7 @@
+ get_event->widgets = (WidgetInfo *)
+ XtCalloc(sizeof(WidgetInfo), get_event->num_entries);
+ for (i = 0; i < get_event->num_entries; i++) {
+- if (!_XEditResGetWidgetInfo(stream, get_event->widgets + i))
++ if (!GetWidgetInfo(stream, get_event->widgets + i))
+ goto done;
+ }
+ }
+@@ -397,7 +430,7 @@
+ _XEditResGet16(stream, &(gv_event->num_entries));
+ gv_event->widgets = (WidgetInfo *)
+ XtCalloc(sizeof(WidgetInfo), gv_event->num_entries);
+- _XEditResGetWidgetInfo(stream, gv_event->widgets);
++ GetWidgetInfo(stream, gv_event->widgets);
+ }
+ break;
+
+@@ -846,7 +879,7 @@
+
+ for (i = 0 ; i < sv_event->num_entries; i++) {
+ if ((str = VerifyWidget(w, &(sv_event->widgets[i]))) != NULL) {
+- _XEditResPutWidgetInfo(stream, &(sv_event->widgets[i]));
++ PutWidgetInfo(stream, &(sv_event->widgets[i]));
+ _XEditResPutString8(stream, str);
+ XtFree(str);
+ count++;
+@@ -973,7 +1006,7 @@
+ */
+
+ (*(info->count))++;
+- _XEditResPutWidgetInfo(info->stream, info->entry);
++ PutWidgetInfo(info->stream, info->entry);
+ _XEditResPutString8(info->stream, pbuf);
+ if (pbuf != buf)
+ XtFree(pbuf);
+@@ -1174,7 +1207,7 @@
+ * Send out the widget id.
+ */
+
+- _XEditResPutWidgetInfo(stream, &(geom_event->widgets[i]));
++ PutWidgetInfo(stream, &(geom_event->widgets[i]));
+ if ((str = VerifyWidget(w, &(geom_event->widgets[i]))) != NULL) {
+ _XEditResPutBool(stream, True); /* an error occured. */
+ _XEditResPutString8(stream, str); /* set message. */
+@@ -1420,7 +1453,7 @@
+ /*
+ * Send out the widget id.
+ */
+- _XEditResPutWidgetInfo(stream, &(res_event->widgets[i]));
++ PutWidgetInfo(stream, &(res_event->widgets[i]));
+ if ((str = VerifyWidget(w, &(res_event->widgets[i]))) != NULL) {
+ _XEditResPutBool(stream, True); /* an error occured. */
+ _XEditResPutString8(stream, str); /* set message. */
+@@ -1766,10 +1799,11 @@
+
+ _XEditResPut16(stream, num_widgets); /* insert number of widgets. */
+ for (i = 0; i < num_widgets; i++) /* insert Widgets themselves. */
+- _XEditResPut32(stream, widget_list[i]);
++ PutWidget(stream, (Widget)widget_list[i]);
+
+ XtFree((char *)widget_list);
+ }
++
+
+ #if 0
+ /************************************************************