Fotoxx: Image editor/viewer

Paint programs, vector editors, 3d modelers, animation editors, etc.
Message
Author
User avatar
technosaurus
Posts: 4853
Joined: Mon 19 May 2008, 01:24
Location: Blue Springs, MO
Contact:

#76 Post by technosaurus »

Here is the patch:

Use

Code: Select all

patch fotoxx-9.8.cpp fotoxx_move_exif_warning.patch
Any thoughts on how this compares to viewnor as a viewer?
Attachments
fotoxx_move_exif_warning.patch.gz
(750 Bytes) Downloaded 1138 times
Check out my [url=https://github.com/technosaurus]github repositories[/url]. I may eventually get around to updating my [url=http://bashismal.blogspot.com]blogspot[/url].

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#77 Post by muggins »

Uploaded v9.8.1( with technosaurus's patch).

Technosaurus, I can't comment on viewnor as I haven't really used it...what are your impressions? Also, did you convey your patch to the developer, as he may be interested? Lastly, where did you dig up those g++ compile optimisations, as they certainly do make a diff?

User avatar
technosaurus
Posts: 4853
Joined: Mon 19 May 2008, 01:24
Location: Blue Springs, MO
Contact:

#78 Post by technosaurus »

The optimizations are just regular size related optimizations (or speed that doesn't add size) from the gcc documentation - mostly referenced for embedded applications elsewhere, but Puppy is basically embedded x86.

I did send the patch in - the dev liked the idea but wants to do something different... give the warning once when any editing of the image is done. Unfortunately I didn't study the code enough to even attempt to help out with that. If I were going to put that much time into the code I would try to patch it to use libexif instead. Oh well - until then - the patch works for my needs.

Viewnior is the defaultimageviewer in Puppy 4.3.X and development has pretty much stopped. Although it is a nice viewer, it seems like Fotoxx would be a good option for smaller puplets to replace mtpaint and viewnior as both the viewer and editor.
Check out my [url=https://github.com/technosaurus]github repositories[/url]. I may eventually get around to updating my [url=http://bashismal.blogspot.com]blogspot[/url].

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#79 Post by muggins »

Thanks technosaurus. I've uploaded v9.9.

User avatar
technosaurus
Posts: 4853
Joined: Mon 19 May 2008, 01:24
Location: Blue Springs, MO
Contact:

#80 Post by technosaurus »

I am impressed - here is the latest changelog - that was fast
fotoxx 9.9

Tonemap gradient method was made easier to use and a little faster. The earlier tonemap functions have been removed because the gradient method is stronger and more flexible than any of them.
Select Area copy and paste has been extended to allow areas to be saved as files and read back later to paste into other images.
Warn about missing program exiftool only when first image edit is initiated.
Bugfix: tonemap sometimes did not work within a selected area.
Check out my [url=https://github.com/technosaurus]github repositories[/url]. I may eventually get around to updating my [url=http://bashismal.blogspot.com]blogspot[/url].

User avatar
ttuuxxx
Posts: 11171
Joined: Sat 05 May 2007, 10:00
Location: Ontario Canada,Sydney Australia
Contact:

#81 Post by ttuuxxx »

The only 2 things I don't like about Fotoxx is that it doesn't batch resize, now that would e a feature and a half, and also when it displays transparent png backgrounds on icons, the background is 100% solid BLACK!!, it should be transparent.
ttuuxxx

bellow is a example fotoxx 9.8.1 VS gpicview , look at the background
also muggins your fotoxx 9.8.1 worked on 2.14x, but 9.9 has a glibC issue and won't run, did you compile it differently? strange that it didn't complain before and now it does.
/usr/bin/fotoxx: /usr/lib/libstdc++.so.6: version `GLIBCXX_3.4.9' not found (required by /usr/bin/fotoxx) <-- V9.9


and when I run the older one you packaged and works fine other than transparencies , I get
sh-3.00# /usr/bin/fotoxx
fotoxx v.9.8.1 2010.03.22
language: en_US
using 1 threads
exiftool sh: exiftool: command not found
xdg-open 1.0.1
sh: ufraw: command not found
sh: exiftool: command not found
exif_get failed


ttuuxxx
Attachments
transparent.jpg
(43.12 KiB) Downloaded 1388 times
http://audio.online-convert.com/ <-- excellent site
http://samples.mplayerhq.hu/A-codecs/ <-- Codec Test Files
http://html5games.com/ <-- excellent HTML5 games :)

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#82 Post by muggins »

Yes, I did compile it differently, using Technosaurus's g++ switches. Funnily though, the fotoxx-9.9.pet package was 3k bigger than if I just used Kornelix's Make. Anyway, I've recompiled using the latter, and re-uploaded...see if that works.

Regarding transparency issue...maybe contact the developer?

User avatar
ttuuxxx
Posts: 11171
Joined: Sat 05 May 2007, 10:00
Location: Ontario Canada,Sydney Australia
Contact:

#83 Post by ttuuxxx »

sorry muggins it didn't work, I'll just upload it here for safe keeping for others who have a glibc version issue.
ttuuxxx
Attachments
fotoxx-9.8.1.pet
(149.32 KiB) Downloaded 791 times
http://audio.online-convert.com/ <-- excellent site
http://samples.mplayerhq.hu/A-codecs/ <-- Codec Test Files
http://html5games.com/ <-- excellent HTML5 games :)

User avatar
technosaurus
Posts: 4853
Joined: Mon 19 May 2008, 01:24
Location: Blue Springs, MO
Contact:

#84 Post by technosaurus »

which set of flags did you use? the bottom set was just a reference base on the standard set - the top set is better for size

Edit - the *sections flags are unnecessary now that freeimage is no longer a dependency - those just helped to eliminate unused sections when compiling against static libfreeimage (but it will make code slightly larger, so it is a gamble as to when to use and when not to)
Check out my [url=https://github.com/technosaurus]github repositories[/url]. I may eventually get around to updating my [url=http://bashismal.blogspot.com]blogspot[/url].

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#85 Post by muggins »

technosaurus:

yes, I used the top set.

ttuxxx:

I tried compiling on p216, which I think has same kernel as p214, but
no luck. Some of the function calls, that fotoxx-9.9 has, don't seem
to be covered by available libs on p216. Or at least AFAIK.

User avatar
technosaurus
Posts: 4853
Joined: Mon 19 May 2008, 01:24
Location: Blue Springs, MO
Contact:

#86 Post by technosaurus »

now that we don't need libfreeimage, this would be better:

Code: Select all

g++ -pipe -combine -Os -momit-leaf-frame-pointer -fomit-frame-pointer -fmerge-all-constants -mpreferred-stack-boundary=2 -march=i386 -mtune=i686 -Wl,-O4,-Os,-relax,--sort-common,--as-needed,-s  -Wall `pkg-config --cflags gtk+-2.0 gthread-2.0` -o fotoxx fotoxx-9.9.cpp zfuncs.cpp -D "DATADIR=\"/usr/share/fotoxx\"" -D "DOCDIR=\"/usr/share/fotoxx\"" -D "BINDIR=\"/usr/bin\"" `pkg-config --libs gtk+-2.0 gthread-2.0` -ltiff
also you can discard my previous patch, since it has been fixed in a better way upstream

installing ufraw and exiftool should remove those errors
Perhaps petget needs a patch for "recommends"?

if the binary is still smaller with the normal method then he has some pretty clean code... still not quite clean enough to use -fwhole-program though, but I think that is because of C++ and the gcc version (when it works, it cuts about 10% off of mtpaint, jwm and the couple other C programs I have tested)
Check out my [url=https://github.com/technosaurus]github repositories[/url]. I may eventually get around to updating my [url=http://bashismal.blogspot.com]blogspot[/url].

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#87 Post by muggins »

Uploaded v10.0.

Yes Technosaurus, you're right, it does compile smaller with the above flags, cheers.

User avatar
ttuuxxx
Posts: 11171
Joined: Sat 05 May 2007, 10:00
Location: Ontario Canada,Sydney Australia
Contact:

#88 Post by ttuuxxx »

lol here's muggins latest with a new folder icon and I removed the locals and docs, much nicer size, works on quirky :)
ttuuxxx
Attachments
fotoxx-10.0.pet
(144.89 KiB) Downloaded 762 times
Last edited by ttuuxxx on Fri 09 Apr 2010, 04:44, edited 1 time in total.
http://audio.online-convert.com/ <-- excellent site
http://samples.mplayerhq.hu/A-codecs/ <-- Codec Test Files
http://html5games.com/ <-- excellent HTML5 games :)

User avatar
technosaurus
Posts: 4853
Joined: Mon 19 May 2008, 01:24
Location: Blue Springs, MO
Contact:

#89 Post by technosaurus »

it contains docs and locales, the binary is only 122kb
Check out my [url=https://github.com/technosaurus]github repositories[/url]. I may eventually get around to updating my [url=http://bashismal.blogspot.com]blogspot[/url].

User avatar
ttuuxxx
Posts: 11171
Joined: Sat 05 May 2007, 10:00
Location: Ontario Canada,Sydney Australia
Contact:

#90 Post by ttuuxxx »

technosaurus wrote:it contains docs and locales, the binary is only 122kb
lol daaaaa should of check before posting, lol
ttuuxxx
http://audio.online-convert.com/ <-- excellent site
http://samples.mplayerhq.hu/A-codecs/ <-- Codec Test Files
http://html5games.com/ <-- excellent HTML5 games :)

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#91 Post by muggins »

Uploaded v10.2.

dawnsboy

#92 Post by dawnsboy »

Hmmm... What to do about the nasty exiv tool warning. Tried the various methods suggested in this thread to no avail.

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#93 Post by muggins »

dawnsboy,

which version fotoxx & which version puppy?

dawnsboy

#94 Post by dawnsboy »

The latest version of fotoxx that you have posted (10.2) on Puppy 4.31.

muggins
Posts: 6724
Joined: Fri 20 Jan 2006, 10:44
Location: hobart

#95 Post by muggins »

dawnsboy,

I mistakenly deleted the original exiftool .pet, from the first page, but I've now re-uploaded it. Running fotoxx, on the commandline in a fresh p431 install, after installing exiftool, ufraw & libgtkimageview, I get:

Code: Select all

# fotoxx
fotoxx v.10.2  2010.04.23
language: en_US 
using 1 threads 
exiftool 7.72
ufraw 0.16
EXIV2 0.16
JPEG enabled.
TIFF enabled.
FITS disabled.
ZIP enabled.
/root/.fotoxx/assigned_tags  - not found 
/root/.fotoxx/tags_index  - not found

Post Reply