[Bug 432205] New: Review Request: exe - eXe eLearning XHTML editor

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




https://bugzilla.redhat.com/show_bug.cgi?id=432205

           Summary: Review Request: exe - eXe eLearning XHTML editor
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: nobody(a)fedoraproject.org
        ReportedBy: j.w.r.degoede(a)hhs.nl
         QAContact: extras-qa(a)fedoraproject.org
                CC: [e-mail],notting(a)redhat.com


Spec URL: http://people.atrpms.net/~hdegoede/exe.spec
SRPM URL: http://people.atrpms.net/~hdegoede/exe-1.03.0.3373-1.fc9.src.rpm
Description:
eXe, the eLearning XHTML editor, is an authoring environment which enables
teachers to publish web content in standard package formats (like IMS
Content Packages and SCORM) without the need to become proficient in HTML
or XML markup.  Content generated using eXe can be used by any Learning
Management System.

---

Note to reviewers, exe uses an included (older) copy of twisted and nevow (which builds on
top of twisted). I've tried making exe use the system version of both, but it will not
work with the system version.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 432205] Review Request: exe - eXe eLearning XHTML editor

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: exe - eXe eLearning XHTML editor


https://bugzilla.redhat.com/show_bug.cgi?id=432205


lkundrak(a)redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody(a)fedoraproject.org    |lkundrak(a)redhat.com
               Flag|                            |fedora-review?




------- Additional Comments From lkundrak(a)redhat.com  2008-03-02 14:35 EST -------
Taking this for review.

(In reply to comment #0)
>  Note to reviewers, exe uses an included (older) copy of twisted and
> nevow 
(which builds on top of twisted). I've tried making exe use the system
version
of both, but it will not work with the system version.

What did it fail for -- isn't it easily fixable? Do you by chance still have the
build logs? Did you communicate that upstream?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 432205] Review Request: exe - eXe eLearning XHTML editor

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: exe - eXe eLearning XHTML editor


https://bugzilla.redhat.com/show_bug.cgi?id=432205





------- Additional Comments From j.w.r.degoede(a)hhs.nl  2008-03-02 16:50 EST -------
(In reply to comment #1)
>  Taking this for review.
>  
>  (In reply to comment #0)
>  > Note to reviewers, exe uses an included (older) copy of twisted and nevow
>  (which builds on top of twisted). I've tried making exe use the system version
>  of both, but it will not work with the system version.
>  
>  What did it fail for -- isn't it easily fixable? 
It builds and runs, but doesn't work. With some python warnings thrown to the
console. Since I'm a bit of a python noob, it isn't easily fixable for me.

Notice that it will only run with the newer bits when replacing both twisted and
nevow, replacing only one is a recipe for even bigger disaster.

>  Did you communicate that upstream? 
Erm, no, will do.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 432205] Review Request: exe - eXe eLearning XHTML editor

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: exe - eXe eLearning XHTML editor


https://bugzilla.redhat.com/show_bug.cgi?id=432205





------- Additional Comments From lkundrak(a)redhat.com  2008-03-15 10:06 EST -------
1.) You install the .desktop file with desktop-file-install;

Please add BuildRequires: desktop-file-utils

2.) Requires:       firefox

This doesn't seem correct to me. It doesn't work with seamonkey or konqueror?
Wouldn't requiring webclient be better?

3.) This could possibly be conditionally present for fedora < 9

%{python_sitelib}/%{name}*.egg-info

4.) RPMLint:

Needs some love; either address those or explain why are they harmless:

exe.noarch: W: devel-file-in-non-devel-package
/usr/share/exe/twisted/internet/iocpreactor/_iocp.c
exe.noarch: W: devel-file-in-non-devel-package
/usr/share/exe/twisted/spread/cBanana.c
exe.noarch: E: arch-independent-package-contains-binary-or-object
/usr/share/exe/templates/mimetex.cgi
exe.noarch: W: class-path-in-manifest /usr/share/exe/templates/geogebra.jar

By the way -- java file? You don't depend on java.

exe.src:37: W: rpm-buildroot-usage %build rm -rf $RPM_BUILD_ROOT
exe.src: E: no-cleaning-of-buildroot %install

Move "rm -rf $RPM_BUILD_ROOT" from %build to %install

5.) Did you check the functionality? The package seemed to be problematic with
my Firefox setup; I couldn't close the window, etc. I'll be more specific if it
worked for you.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 432205] Review Request: exe - eXe eLearning XHTML editor

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: exe - eXe eLearning XHTML editor


https://bugzilla.redhat.com/show_bug.cgi?id=432205





------- Additional Comments From j.w.r.degoede(a)hhs.nl  2008-03-16 10:18 EST -------
(In reply to comment #3)
>  1.) You install the .desktop file with desktop-file-install;
>  
>  Please add BuildRequires: desktop-file-utils
>   
Will do (once other points below are clear).

>  2.) Requires:       firefox
>  
>  This doesn't seem correct to me. It doesn't work with seamonkey or konqueror?
>  Wouldn't requiring webclient be better?
>   
No it needs firefox (maybe now xulrunner) its an application written in Xul,
deriving from the firefox xul code, some error dialogs etc, still clearly show this.

>  3.) This could possibly be conditionally present for fedora < 9
>  
>  %{python_sitelib}/%{name}*.egg-info
>   
Good point.


>  4.) RPMLint:
>  
>  Needs some love; either address those or explain why are they harmless:
>  
>  exe.noarch: W: devel-file-in-non-devel-package
>  /usr/share/exe/twisted/internet/iocpreactor/_iocp.c
>  exe.noarch: W: devel-file-in-non-devel-package
>  /usr/share/exe/twisted/spread/cBanana.c 
I'll fix these, it looks like the included copy of twisted is a full copy, I'll
strip it down to only the needed parts.

>  exe.noarch: E: arch-independent-package-contains-binary-or-object
>  /usr/share/exe/templates/mimetex.cgi 
Hmm, how weird, I'll ask upstream why thats there (and in the mean time remove it).

>  exe.noarch: W: class-path-in-manifest
> /usr/share/exe/templates/geogebra.jar
>  
>  By the way -- java file? You don't depend on java.
>   
Yes, exe is an editor for eleanring material, which is meant to be viewed with a
webbrowser, I guess this is an applet for one of the elearning template's which
are part of one eXe.

>  exe.src:37: W: rpm-buildroot-usage %build rm -rf $RPM_BUILD_ROOT
>  exe.src: E: no-cleaning-of-buildroot %install
>  
>  Move "rm -rf $RPM_BUILD_ROOT" from %build to %install
>   
Will do.

>  5.) Did you check the functionality? The package seemed to be
> problematic with
>  my Firefox setup; I couldn't close the window, etc. I'll be more specific if it
>  worked for you. 
I packaged this because a college of mine asked for it (I recently converted him
from Ubuntu to Fedora), he has been using it for a while. The window not closing
part is "normal", you should get a dialoge saying that you should use the menu
to exit instead though, do you get that dialog?


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 432205] Review Request: exe - eXe eLearning XHTML editor

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: exe - eXe eLearning XHTML editor


https://bugzilla.redhat.com/show_bug.cgi?id=432205





------- Additional Comments From j.w.r.degoede(a)hhs.nl  2008-03-17 05:28 EST -------
I've done some more looking into the inclusing of cgi and jar files in eXe and
I'm currently working on this with upstream, see:
https://eduforge.org/forum/forum.php?thread_id=2150&forum_id=298


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.