[needs-packaging] Art of Illusion - Easy to use 3D modeling studio

Bug #185810 reported by Marc Carson on 2008-01-25
58
This bug affects 11 people
Affects Status Importance Assigned to Milestone
Debian
Fix Released
Unknown
Ubuntu
Wishlist
Unassigned

Bug Description

(Needs packaging)

Art of Illusion is a 3D modeling, animation, and rendering studio geared toward users who want a clean, easy-to-use interface or who are new to 3D design. The software is licensed under the GPL and has been available for around seven years. It is currently on version 2.5.1 and development is quite stable and active.

The source code can be downloaded via the following URL, in the Downloads section

URL: http://www.artofillusion.org
Wikipedia: http://en.wikipedia.org/wiki/Art_of_Illusion
Source code: http://sourceforge.net/project/showfiles.php?group_id=15063

Not in Ubuntu or Debian - Confirmed.

Changed in debian:
status: Unknown → New
Changed in debian:
status: New → Fix Released

There is no fix released in Debian. The bug was automatically closed because of 18 months inactivity.

I have packaged it into a PPA.

To install Art of Illusion on your computer do:

sudo add-apt-repository ppa:altair-ibn-la-ahad/artofillusion
sudo apt-get update
sudo apt-get install artofillusion

run the program with:

artofillusion

Changed in ubuntu:
assignee: nobody → Andreas Altaïr Redmer (altair-ibn-la-ahad)
status: Confirmed → In Progress
Daniel Holbach (dholbach) wrote :

A few comments when reviewing the package:
 - could artofillusion.sh and build.xml be made part of a patch (in debian/patches) or somehow else placed in the debian/ directory?
 - Could you make "(Closes: LP:#185810)" be "(LP: #185810)" in debian/changelog?
 - you could set debian/source/format to "3.0 (quilt)"
 - can you have a closer look at the licenses of the files again? some are GPL (v2), others are GPL (v2 or later)
   This snippet could be useful: for i in `find . -type f`; do licensecheck $i; done
 - current Standards-Version is 3.9.5
 - a debian/watch would be nice to have
 - The synopsis (first line in the package "Description:" field, the short description) ends with a full stop "." character.
 - empty directories
   usr/share/lib/artofillusion/HelpPlugin/AOIHelp/helpset/FurtherHelp/images/
   usr/share/lib/artofillusion/HelpPlugin/AOIHelp/helpset/HowTo/images/
   usr/share/lib/artofillusion/HelpPlugin/AOIHelp/helpset/Plugins/Help/
   usr/share/lib/artofillusion/HelpPlugin/AOIHelp/helpset/Problems/
   usr/share/lib/artofillusion/HelpPlugin/docs/
   usr/share/lib/artofillusion/Scripts/Objects/
   usr/share/lib/artofillusion/Scripts/Startup/
   usr/share/lib/artofillusion/Scripts/Tools/
 - maybe an artofillusion-common (or -data) (arch: all) package would be useful to ship everything that's in usr/share (16M)
 - manpage for usr/bin/artofillusion would be nice to have
 - LICENSE contains lots of ^M chars

Of the above the debian/copyright review is the most important. A lot of other things are nice to have.

Hi. Thanks a lot for the input. I was able to improve the package and re-upload it. In detail:

1. upstream is now untouched - artofillusion.sh and build.xml are in debian/patches
2. changelog is fixed, current Standards-Version fixed, synopsis fixed
3. format is now 3.0 (quilt) and whole package is quilted.
4. Thanks to your advice (licencecheck) I was able to add some missing authors and put all the GPL2+ in the proper section
5. manpage added
6. LICENCE file fixed (also in debian/patches)
7. package split into artofillusion-common and artofillusion
8. package is now Lintian-clean

-----

Remaining issues:

1. debian/watch added: but it doesn't work. The upstream uses SF and a wierd versioning. Version 3.0 is in the filename encoded as "30" and version 2.9.2 as "292". As a result it always reports that 292 is the newest version. What can we do there?

2. empty directories: without the empty directories, the program runs into errors. You can test that by deleting them after installation. I could remove them from the build and add them to artofillusion.sh, so they will be created on the first start. What shall we do?

If you have time I would be thankful, if you can review it again. Thanks.

Daniel Holbach (dholbach) wrote :

Thanks for looking into this. I'll take a look at the package in a bit.

Regarding the empty directories: Can we be sure that no files are supposed to be shipped in these directories?

Daniel Holbach (dholbach) wrote :

Regarding debian/watch: Assuming that "xyz" will always indicate "x.y.z" the following might work:

version=3
opts="uversionmangle=s/(\d)/$1\./g;s/\.$//" \
 http://sf.net/aoi/aoisrc(\d\S*)\.(?:zip|tgz|tbz|txz|(?:tar\.(?:gz|bz2|xz)))

"uscan --verbose" seemed to do the right thing though.

Daniel Holbach (dholbach) wrote :

Another small thing:

duplicate-short-description artofillusion artofillusion-common

It might help to add something like "<text> - data files" or some such.

Daniel Holbach (dholbach) wrote :

Apart from that: good work!

Thanks. I updated and improved it again:

1. duplicate-short-description fixed
2. debian/watch version fixed (your fix worked)
3. Regarding the empty directories:

I doublechecked that again and did some more tests. The 3 empty directories */Scripts/* are always empty but the program needs them to run.

The other empty directories (*/HelpPlugin/*) are also always empty. There is nothing to be shipped there. They appear to be a test, or some half-future-features. I also checked the help in the installed program and all the images are complete. Also I could remove these empty dirs from the package because they had no effect on the installed program. So I did that.

I can confirm, that there is nothing to be shipped into the empty directories.

Daniel Holbach (dholbach) wrote :

Hum... how about the bundled .jar files? Could they be replaced with symlinks to system versions or (if not available) be built from source?

I know this massively complicates things, but it'd be good to know which parts come from where and why they're bundled. I guess the archive admins are going to ask the same thing.

Hi. I must apoligize. I didn't see that there are prebuilt jar files included. Actually I thought there would be a warning for that like "Rejected, because you tried to upload bkinary code in a source package".

I aggree that they must be taken out there and most of them are actually libs in existing debian java packages.

I'm working on it.

Daniel Holbach (dholbach) wrote :

For now I'll unsubscribe the sponsors team, please resubscribe once you need some reviewing done. Thanks.

I could link most of the binary JARs that are included in the source package, to existing JARs in Debian packages. But there are 2 causing problems:

1. is a small GUI library that the upstream author releases by himself in a separate upstream. But it is not really compilable and there is no hint which exact version the author compiled and included into the source.tar.gz . That appears to be the reason, why it is in there (it is kinda messy).

2. is a SWF creation library, which I found not to be open source. The package could also be created without this library and some patches to the source, to deactivate thew SWF export. But that is to complicated for me, since the source code is a bit confusing.

I'm discontinuing my work on this package, since the upstream is in poor quality. It might be senseful to pick this up in future again.

The PPA I created is working and can be used to install and use the software. After the review the package is also already in a good quality (except the existing JARs in the source). All my work is in the linked BZR branch, so in easily be continued.

Changed in ubuntu:
assignee: Andreas Altaïr Redmer (altair-ibn-la-ahad) → nobody
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.