Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 204832 (wxMaxima) - Review Request: wxMaxima - wxWidgets interface for maxima
Summary: Review Request: wxMaxima - wxWidgets interface for maxima
Keywords:
Status: CLOSED NEXTRELEASE
Alias: wxMaxima
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Gérard Milmeister
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-08-31 19:30 UTC by Rex Dieter
Modified: 2014-12-17 01:38 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-09-07 18:40:27 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Rex Dieter 2006-08-31 19:30:56 UTC
Spec Name or Url: http://apt.kde-redhat.org/apt/fedora/SPECS/wxMaxima.spec
SRPM Name or Url: http://apt.kde-redhat.org/apt/fedora/all/SRPMS.stable/wxMaxima-0.7.0-1.src.rpm
Description: 
wxMaxima is a wxWidgets interface for the computer algebra system Maxima.

Comment 1 Gérard Milmeister 2006-09-04 20:12:05 UTC
* builds fine in mock (FC5)
* rename maxima-new.png to wxmaxima.png for clarity
* COPYING and README are not necessary in /usr/share/wxMaxima
* in the .desktop file:
  I think that there should be no X-Red-Hat* categories, but I may
  be wrong
* I would change the description and summary a little, for example
  summary: Graphical user interface for Maxima
  description: A Graphical user interface for the computer algebra
  system Maxima using wxWidgets.

  This way, we also avoid starting the description with a lower
  case letter :-) Also, it emphasizes the "user interface", since
  the fact that it uses wxWidgets is not that important for most
  possible. Also the original text gives the impression that it
  was about wxWidgets bindings...
* Help doesn't work:
  Cannot open HTML help book: /usr/share/maxima/5.9.3/doc/html/header.hhp
  Is there a .hhp help file somewhere in the maxima distribution?

Comment 2 Rex Dieter 2006-09-05 12:52:59 UTC
>  I think that there should be no X-Red-Hat* categories

I agree.

> * Help doesn't work:
I'd venture to guess this bit may require maxima-5.9.3.99 (or better).

Comment 3 Rex Dieter 2006-09-05 13:08:21 UTC
Spec Name or Url: http://apt.kde-redhat.org/apt/fedora/SPECS/wxMaxima.spec
SRPM Name or Url:
http://apt.kde-redhat.org/apt/fedora/all/SRPMS.stable/wxMaxima-0.7.0-2.src.rpm

%changelog
* Tue Sep 05 2006 Rex Dieter <rexdieter[AT]users.sf.net> 0.7.0-2
- update %%description, %%summary
- rename icon -> wxmaxima.png
- omit extraneous COPYING, README
- .desktop: remove X-Red-Hat* categories

Comment 4 Gérard Milmeister 2006-09-05 15:05:23 UTC
(In reply to comment #2)
> >  I think that there should be no X-Red-Hat* categories
> 
> I agree.
> 
> > * Help doesn't work:
> I'd venture to guess this bit may require maxima-5.9.3.99 (or better).
I see that this version is in FE development. Would you building
it for FE5?


Comment 5 Rex Dieter 2006-09-05 15:09:38 UTC
>>> * Help doesn't work:
>> I'd venture to guess this bit may require maxima-5.9.3.99 (or better).

> I see that this version is in FE development. Would you building
> it for FE5?

I've been waiting for 5.9.4(final) before pushing a maxima update to any
non-devel repo. 

Comment 6 Gérard Milmeister 2006-09-06 20:47:45 UTC
-> In the description there is a duplicate "the the".
Otherwise everything is fine.
Builds in mock FC4 and devel.
rpmlint is silent.
APPROVED

Comment 7 Jason Tibbitts 2006-09-07 00:37:42 UTC
Umm, this bug is still NEW, is still blocking FE-NEW and is not assigned to
anyone.  Why is someone approving it?

Comment 8 Rex Dieter 2006-09-07 18:40:27 UTC
typo fixed, imported, +owners.list, built for devel.


Note You need to log in before you can comment on or make changes to this bug.