Bug 2212

Summary: booh missing from mageia 1
Product: Mageia Reporter: Guillaume Rousse <guillomovitch>
Component: New RPM package requestAssignee: QA Team <qa-bugs>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal CC: davidwhodgins, derekjenn, dmorganec, guillomovitch, stormi-mageia, sysadmin-bugs
Version: 1Keywords: validated_update
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: booh CVE:
Status comment:

Description Guillaume Rousse 2011-07-19 22:53:24 CEST
booh is missing from mageia 1. I just submitted booh 0.9.3 to core/updates_testing. Please test and validate as update.
Comment 1 Guillaume Rousse 2011-07-19 23:03:30 CEST
Advisory:
This update adds the booh web album generator which was present in Mandriva 2010.2 but missing in Mageia 1.
Comment 2 Dave Hodgins 2011-07-23 01:34:32 CEST
I'm probably doing something wrong.

I used booh-classifier to add labels to one photo, and one vidio.

When I try to booh to create a new album, the konsole I ran it from shows
booh-backend --source '/tmp/tmp/cult' --destination '/tmp/tmp/booh' --config-skel '/home/dave/.booh/cult' --for-gui --verbose-level 2 --theme simple --sizes medium,large --thumbnails-per-row 4  --made-with 'made with <a href=%booh>booh</a>!' --index-link ''   --comments-format '%t' --flv-generator '/usr/bin/ffmpeg -i %f -b 800k -ar 22050 -ab 32k %o'  --info-pipe /home/dave/tmp/boohpipe20110722-27535-1aym678-0
/usr/bin/booh-backend:28:in `require': no such file to load -- gettext/locale (LoadError)
        from /usr/bin/booh-backend:28

and the dialog displays "something went wrong".

CC: (none) => davidwhodgins

Comment 3 Dave Hodgins 2011-07-25 00:08:07 CEST
I successfully created a web album with booh-0.9.3-3, on my i586 system.

I did find one serious problem though.

When I was selecting the directory to store the album in, if I just highlighted
the directory name, and then selected open, it worked.

If I double clicked the directory name (which doesn't contain anything), and
then selected Open, the ruby process goes to nearly 100% cpu usage, and strace
doesn't show any system calls being made.

After clicking on the close button, kwin can be used to terminate the application.

As the program is still usable, if care is taken when selecting the directory
(single click, instead of double click), I think the update should go forward,
but the bug should be reported upstream.

My guess is that this is a ruby bug, rather then a booh bug.
Comment 4 Derek Jennings 2011-07-26 16:04:13 CEST
Tests OK on x86_64 apart from the same issue noted in Comment 3 when double clicking on a storage directory.

CC: (none) => derekjenn

Comment 5 Samuel Verschelde 2011-07-26 16:22:50 CEST
The bug in comment #3 is a serious bug because it's not only a non-functional feature but it takes the CPU usage to 100%. I would prefer if we could find a fix before pushing the update.

CC: (none) => stormi

Comment 6 Dave Hodgins 2011-07-27 00:19:37 CEST
I've never even looked at a ruby program before. :-)
After reading a bit about it, I added some puts statements, to see where
it's hanging.

It's line 3446 of /usr/bin/booh, which has
if fc.run == Gtk::Dialog::RESPONSE_ACCEPT

So it seems to be a problem with Gtk::FileChooserDialog

What's really strange, is the code for selecting the directory with the pictures
is the same, yet it doesn't show the same problem.

Any ruby/gtk experts here?
Comment 7 Guillaume Rousse 2011-08-22 14:03:26 CEST
You're mixing the issues here. booh may have its own problems, but blocking its availability for stable release won't help fixing it.
Comment 8 D Morgan 2011-08-22 14:34:47 CEST
yes i think we can push it, but work on the "bug" after.

CC: (none) => dmorganec

Comment 9 Samuel Verschelde 2011-08-22 14:59:50 CEST
Having the packager showing he's willing to search for solutions is a key consideration for pushing new packages. We can't push unsupported things just because they were not there and we want them. Thus that was not irrelevant to point out problems. It doesn't mean bugs should always block pushing of new packages, but what we really need is an available packager to discuss such issues. Now, if you want to push it in its buggy state, push it.
Comment 10 Guillaume Rousse 2011-08-22 15:06:21 CEST
Sorry for not being responding enough to your taste. As actually I don't really care about this issue, let's close it now.

Status: NEW => RESOLVED
Resolution: (none) => WONTFIX

Comment 11 Samuel Verschelde 2011-08-22 15:37:14 CEST
I was trying to answer in a constructive way, but probably failed.

Now, reopening in case you change your mind or in case there's another packager ready to maintain this package for mageia 1 (looking after the bugs before or after the update push. Ideally looking at them before, and depending on the amount of work, maybe fixing them after).

Status: RESOLVED => REOPENED
Resolution: WONTFIX => (none)

Comment 12 Samuel Verschelde 2011-09-10 01:54:49 CEST
Ok, I'll suppose that Guillaume's closing the door was not related to an unwillingness to fix the bug but rather to my comment #9, and that the bug will be fixed afterwards.

Please push booh from Core Updates Testing to Core Updates

Advisory:
This update adds the booh web album generator which was present in Mandriva
2010.2 but missing in Mageia 1.

Keywords: (none) => validated_update
CC: (none) => guillomovitch, sysadmin-bugs

Comment 13 D Morgan 2011-09-14 08:51:50 CEST
update pushed.

Status: REOPENED => RESOLVED
Resolution: (none) => FIXED