Bug 193772

Summary:	Review Request: Xbae - Matrix,Caption,Input widgets for Motif
Product:	[Fedora] Fedora	Reporter:	Tobias Oed <tobiasoed>
Component:	Package Review	Assignee:	Thorsten Leemhuis (ignored mailbox) <bugzilla-sink>
Status:	CLOSED DUPLICATE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	pertusus
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-06-01 16:19:08 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Tobias Oed 2006-06-01 14:36:26 UTC

Spec URL: http://prdownloads.sourceforge.net/xbae/Xbae.spec
SRPM URL: http://prdownloads.sourceforge.net/xbae/Xbae-4.60.4-1.src.rpm
Description: 
The Xbae widget set was in fedora 1, and dropped in fedora 2. I repackaged it for inclusion in fedora 5-extras.

This package provides the Motif matrix and caption. The matrix widget
is widely used and will display up to four billion rows of data in a
spreadsheet-like form. The caption widget is a simple Motif manager
widget that associates a label with a child. You will need to have
either Motif or LessTif installed in order to use Xbae's widgets.

Comment 1 Patrice Dumas 2006-06-01 15:59:52 UTC

I have allready proposed Xbae here:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192418

I think my proposal is a bit better since it is simple and also 
package the examples. But I should update to the latest version.
If you really want to maintain Xbae, no problem I'll do a patch
to your spec.

Also I depend on Wcl to verify that the corresponding example 
builds.

Comment 2 Tobias Oed 2006-06-01 16:19:08 UTC

I don't really want to maintain the spec file, I already work on the sources :)
I'll gladly let you take care of this, and mark this report as a duplicate of yours.
wrt. the examples, I think you may want to split them out in xbae-examples if
that's even possible. (I don't know rpm all that well).
Cheers, Tobias.


*** This bug has been marked as a duplicate of 192418 ***

Comment 3 Patrice Dumas 2006-06-01 16:28:41 UTC

Currently I package them in xbae-devel. I don't think they deserve
a separate subpackage.

Comment 4 Patrice Dumas 2006-06-01 22:03:02 UTC

I cannot find anything packaged in 

%{_prefix}/share/aclocal/*

in your specfile. Is it an error? I ask because this is something
I could take from your spec, but I cannot find it.

Comment 5 Tobias Oed 2006-06-01 22:55:22 UTC

That's wierd, my devel rpm includes 
/usr/share/aclocal/ac_find_xbae.m4 5618 1149167337
34839c3fd5b8d6d7d6dc693d4c2ddce9 0100644 root root 0 0 0 X
which turns out to be a copy of the ac_find_xbae.m4 that's at the top of the
source tree. Part of the patch I include in the rpm is supposed to make that one
get installed.
Tobias.

Comment 6 Patrice Dumas 2006-06-02 07:37:02 UTC

Seems to me that it is not part of your patch but it is in 
configure... Your patch avoids documentation installation.
I solved that issue differently. 

However aclocal isn't found by configure on my box, I have
to investigate that.

Comment 7 Tobias Oed 2006-06-02 10:14:00 UTC

These two bits in my patch:
In Makefile.am
-
-if Aclocal
-
 aclocal_DATA = ac_find_xbae.m4

-endif
and Makefine.in
-@Aclocal_TRUE@aclocal_DATA = ac_find_xbae.m4
+aclocal_DATA = ac_find_xbae.m4
I do install the man pages as documentation with -devlel. The html stuff that
comes with xbae is way out of date, so I decided to leave it out.
Tobias.

Comment 8 Patrice Dumas 2006-06-02 14:39:02 UTC

Strangely this doesnt seems to be that patch which is in your
.src.rpm... I can do the same however.