[patch] Inactive import folder in f-spot

Bug #293305 reported by demas
66
This bug affects 4 people
Affects Status Importance Assigned to Milestone
f-spot (Debian)
Fix Released
Unknown
f-spot (Ubuntu)
Fix Released
High
Chris Coulson

Bug Description

Binary package hint: f-spot

F-spot version: f-spot 0.5.0.3
Ubuntu version: 8.10
Locale: ru_RU.UTF-8

Steps:

1. Launch f-spot
2. Edit -> Preferences -> Import settings -> Folder. Combo-box is gray and inactive.

Expected:

Box is active and available.

Notes:

1. If I launch f-spot from command line like this:

      LC_ALL=C LANG=C f-spot

     The problem disappears.

2. I have reported about this problem in IRC channel and I have got this patch:

 http://monoport.com/38016

Revision history for this message
Florian Heinle (tiax) wrote :

After running f-spot with LC_ALL=C LANG=C f-spot the box is enabled and it's possible to set the import path, which will then get stored correctly to gconf (workaround)

Stephane Delcroix wrote a patch against 0.5.0.3, which is attached

description: updated
Revision history for this message
Greg Grossmeier (greg.grossmeier) wrote :

According to one of the upstream developers (sde) this happens to all non-english locales, hence the High importance.

Changed in f-spot:
importance: Undecided → High
status: New → Triaged
Revision history for this message
Florian Heinle (tiax) wrote :

this is the right patch. It's the same as linked above. I don't trust pastebins, let's host it on launchpad.

Revision history for this message
svenmeier (sven-meiers) wrote :

Thanks for the workaround.

Revision history for this message
Janne Moren (jan-moren-gmail) wrote :

Is there a not-too-painful way to move images that's already been imported and edited in the wrong folder?

Revision history for this message
demas (andrey-demidov-gmail) wrote :

Hmm....

1. Just run once that in command line: LC_ALL=C LANG=C f-spot
2. Set the folder
3. Exit F-Spot
4. Launch F-Spot and enjoy.

Changed in f-spot:
assignee: nobody → desktop-bugs
Revision history for this message
Martin Reiche (grimsrud) wrote :

Problem persists in Jaunty Alpha 3 with all updates installed as of today.

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Yeah, F-Spot is still the same upstream version in Jaunty, so this will still be a problem. Thanks for digging out the patch Florian. I've subscribed ubuntu-main-sponsors to this so your patch can be uploaded in to Jaunty.

Revision history for this message
Sebastien Bacher (seb128) wrote :

does anybody want to work on a debdiff for this update and maybe rebase the upgrade on the current debian version?

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

I can work on that this afternoon

Changed in f-spot:
assignee: desktop-bugs → chrisccoulson
status: Triaged → In Progress
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

I've merged it with 0.5.0.3-1 in Debian experimental. Here is the 0.5.0.3-0ubuntu5 -> 0.5.0.3-1ubuntu1 debdiff

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Try again!

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Here is the delta to Debian

Changed in f-spot:
status: In Progress → Triaged
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Note that this is currently blocked by bug 324741 (this depends on the new mono-addins)

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

I sent this to Debian. If the patch gets uploaded there, we can just sync it instead

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

The patch (and the remaining delta) has been committed to Experimental, and is just waiting to be uploaded:

http://svn.debian.org/viewsvn/pkg-cli-apps?rev=4515&sortby=rev&view=rev

Revision history for this message
Sebastien Bacher (seb128) wrote :

thank you for your work there, will sponsor the change later and sync the new revision when it's uploaded to debian

Changed in f-spot:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package f-spot - 0.5.0.3-1ubuntu1

---------------
f-spot (0.5.0.3-1ubuntu1) jaunty; urgency=low

  * Merged from Debian experimental, remaining changes:
    - debian/patches/svn-r4545_locales-import.dpatch:
      - Patch backported from SVN trunk revision 4545 -
        initialize the translation catalog earlier (LP: #293305).
        Thanks to Florian Heinle for finding the patch.
    - debian/control:
      - Build-depend on libmono-dev (>= 1.2.4).

 -- Chris Coulson <email address hidden> Fri, 06 Feb 2009 17:06:37 +0000

Changed in f-spot:
status: Fix Committed → Fix Released
Revision history for this message
Iain Lane (laney) wrote :
Revision history for this message
Andrea Ratto (andrearatto) wrote :

Is it going to be backported or updated in intrepid too?

Changed in f-spot:
status: Unknown → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

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