---------- Forwarded message ---------- From: GIMP
Date: Thu, Dec 24, 2009 at 3:04 PM
Subject: [Bug 605366] OpenRaster support To: jononor@gmail.com

https://bugzilla.gnome.org/show_bug.cgi?id=605366 GIMP | General | unspecified

Martin Nordholts changed:

What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW CC| |enselic@gmail.com Target Milestone|--- |2.10 Ever Confirmed|0 |1

--- Comment #1 from Martin Nordholts 2009-12-24 14:03:58 UTC ---
The best way to get OpenRaster into GIMP is better discussed on the gimp-developer mailing list, but I think GIMP 2.10/3.0 will be a good time to
seriously look into this.

-----------------

Picking up this right now. As stated in the report, there is a Python input handler plug-in (written by me) available that provides OpenRaster support. Are you be interested in shipping this for 2.8? What would it take?

Obviously that solution is not good enough in the long term. So what would be the best approach for a long term solution? Right now, I see two viable alternatives:
1) Making a separate library thats suitable for use from C that handles the low-level/OpenRaster specific stuff. MyPaint and other apps could possibly use this as well.
2) Implement support directly in GIMP. Possibly using GEGL(?)

On Thu, Dec 24, 2009 at 4:47 PM, Martin Nordholts wrote:

Jon Nordby wrote:

Picking up this right now. As stated in the report, there is a Python input handler plug-in (written by me) available that provides OpenRaster support. Are you be interested in shipping this for 2.8? What would it take?

Personally I think we should accept a sane patch that gives GIMP the ability to load OpenRaster files, for 2.8.

Please test, review and give feedback on the following patch/plug-in. On any aspect. It gives GIMP the ability to load and save OpenRaster files. Tested with GIMP 2.6.8 and current git (f0348aff40b27) using Python 2.6. Should work equally well with python 2.5 and maybe also 2.4.

Some questions: - The thumbnail function does not seem to be called (at least a print statement does not give output) but thumbnailing does work as expected. Anyone got an explanation?
- Should the plug-in set mime-type association on the saving function somehow?

Known limitations:
- Nested layers are flattened instead of leveraging nested layers available in 2.7+git
- Thumbnail scaling is not implemented

On Mon, Dec 28, 2009 at 12:39 PM, Sven Neumann wrote:

On Sun, 2009-12-27 at 18:59 +0100, Jon Nordby wrote:

Please test, review and give feedback on the following patch/plug-in. On any aspect. It gives GIMP the ability to load and save OpenRaster files.

Very nice indeed. Would you mind renaming the file and procedures to file-openraster-* ?

No a problem. Any other feedback? Anything; code or naming conventions, code clarity, issues, et.c.? Is there anything else keeping this from being accepted?

And can anyone answer my questions in my previous mail? "- The thumbnail function does not seem to be called (at least a print statement does not give output) but thumbnailing does work as expected.* Anyone got an explanation?
- Should the plug-in set mime-type association on the saving function somehow?"

* In the "open file" dialog that is.

Happy new years to all!

I've updated the patch to incorporate Svens feedback; changed the file and procedures to file-openraster-*

Hoping this can be committed now. Also, please consider shipping it with the next 2.6 stable.