The time now is Wed 22 May 2013, 10:12
All times are UTC - 4 |
| Author |
Message |
technosaurus

Joined: 18 May 2008 Posts: 3843
|
Posted: Sun 21 Mar 2010, 18:29 Post subject:
|
|
Here is the patch:
Use
| Code: | | patch fotoxx-9.8.cpp fotoxx_move_exif_warning.patch |
Any thoughts on how this compares to viewnor as a viewer?
| Description |
|

Download |
| Filename |
fotoxx_move_exif_warning.patch.gz |
| Filesize |
750 Bytes |
| Downloaded |
442 Time(s) |
_________________ Puppy Web Desktop Now with pet packages - Pet Packaging 100 & 101
|
|
Back to top
|
|
 |
muggins
Joined: 20 Jan 2006 Posts: 6660 Location: lisbon
|
Posted: Thu 25 Mar 2010, 03:01 Post subject:
|
|
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?
|
|
Back to top
|
|
 |
technosaurus

Joined: 18 May 2008 Posts: 3843
|
Posted: Thu 25 Mar 2010, 16:49 Post subject:
|
|
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.
_________________ Puppy Web Desktop Now with pet packages - Pet Packaging 100 & 101
|
|
Back to top
|
|
 |
muggins
Joined: 20 Jan 2006 Posts: 6660 Location: lisbon
|
Posted: Mon 29 Mar 2010, 20:52 Post subject:
|
|
Thanks technosaurus. I've uploaded v9.9.
|
|
Back to top
|
|
 |
technosaurus

Joined: 18 May 2008 Posts: 3843
|
Posted: Tue 30 Mar 2010, 15:53 Post subject:
|
|
I am impressed - here is the latest changelog - that was fast
| Quote: | 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. |
_________________ Puppy Web Desktop Now with pet packages - Pet Packaging 100 & 101
|
|
Back to top
|
|
 |
ttuuxxx

Joined: 05 May 2007 Posts: 10720 Location: Ontario Canada,Sydney Australia
|
Posted: Tue 30 Mar 2010, 19:38 Post subject:
|
|
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
| Description |
|
| Filesize |
43.12 KB |
| Viewed |
1112 Time(s) |

|
_________________ http://audio.online-convert.com/ <-- excellent site
http://samples.mplayerhq.hu/A-codecs/ <-- Codec Test Files
http://html5games.com/ <-- excellent HTML5 games

|
|
Back to top
|
|
 |
muggins
Joined: 20 Jan 2006 Posts: 6660 Location: lisbon
|
Posted: Tue 30 Mar 2010, 21:01 Post subject:
|
|
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?
|
|
Back to top
|
|
 |
ttuuxxx

Joined: 05 May 2007 Posts: 10720 Location: Ontario Canada,Sydney Australia
|
Posted: Tue 30 Mar 2010, 21:49 Post subject:
|
|
sorry muggins it didn't work, I'll just upload it here for safe keeping for others who have a glibc version issue.
ttuuxxx
| Description |
|

Download |
| Filename |
fotoxx-9.8.1.pet |
| Filesize |
149.32 KB |
| Downloaded |
440 Time(s) |
_________________ http://audio.online-convert.com/ <-- excellent site
http://samples.mplayerhq.hu/A-codecs/ <-- Codec Test Files
http://html5games.com/ <-- excellent HTML5 games

|
|
Back to top
|
|
 |
technosaurus

Joined: 18 May 2008 Posts: 3843
|
Posted: Wed 31 Mar 2010, 02:00 Post subject:
|
|
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)
_________________ Puppy Web Desktop Now with pet packages - Pet Packaging 100 & 101
|
|
Back to top
|
|
 |
muggins
Joined: 20 Jan 2006 Posts: 6660 Location: lisbon
|
Posted: Thu 01 Apr 2010, 23:24 Post subject:
|
|
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.
|
|
Back to top
|
|
 |
technosaurus

Joined: 18 May 2008 Posts: 3843
|
Posted: Fri 02 Apr 2010, 09:05 Post subject:
|
|
now that we don't need libfreeimage, this would be better:
| Code: | | 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)
_________________ Puppy Web Desktop Now with pet packages - Pet Packaging 100 & 101
|
|
Back to top
|
|
 |
muggins
Joined: 20 Jan 2006 Posts: 6660 Location: lisbon
|
Posted: Thu 08 Apr 2010, 19:47 Post subject:
|
|
Uploaded v10.0.
Yes Technosaurus, you're right, it does compile smaller with the above flags, cheers.
|
|
Back to top
|
|
 |
ttuuxxx

Joined: 05 May 2007 Posts: 10720 Location: Ontario Canada,Sydney Australia
|
Posted: Thu 08 Apr 2010, 23:03 Post subject:
|
|
lol here's muggins latest with a new folder icon and I removed the locals and docs, much nicer size, works on quirky
ttuuxxx
| Description |
|

Download |
| Filename |
fotoxx-10.0.pet |
| Filesize |
144.89 KB |
| Downloaded |
425 Time(s) |
_________________ http://audio.online-convert.com/ <-- excellent site
http://samples.mplayerhq.hu/A-codecs/ <-- Codec Test Files
http://html5games.com/ <-- excellent HTML5 games

Last edited by ttuuxxx on Fri 09 Apr 2010, 00:44; edited 1 time in total
|
|
Back to top
|
|
 |
technosaurus

Joined: 18 May 2008 Posts: 3843
|
Posted: Thu 08 Apr 2010, 23:10 Post subject:
|
|
it contains docs and locales, the binary is only 122kb
_________________ Puppy Web Desktop Now with pet packages - Pet Packaging 100 & 101
|
|
Back to top
|
|
 |
ttuuxxx

Joined: 05 May 2007 Posts: 10720 Location: Ontario Canada,Sydney Australia
|
Posted: Fri 09 Apr 2010, 00:45 Post subject:
|
|
| 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

|
|
Back to top
|
|
 |
|
|
|
You cannot post new topics in this forum You cannot reply to topics in this forum You cannot edit your posts in this forum You cannot delete your posts in this forum You cannot vote in polls in this forum You cannot attach files in this forum You can download files in this forum
|
Powered by phpBB © 2001, 2005 phpBB Group
|