[patch] Inactive import folder in f-spot

Bug #293305 reported by demas on 2008-11-03
66
This bug affects 4 people
Affects Status Importance Assigned to Milestone
f-spot (Debian)
Fix Released
Unknown
f-spot (Ubuntu)
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

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

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
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.

svenmeier (sven-meiers) wrote :

Thanks for the workaround.

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?

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
Martin Reiche (grimsrud) wrote :

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

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.

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?

Chris Coulson (chrisccoulson) wrote :

I can work on that this afternoon

Changed in f-spot:
assignee: desktop-bugs → chrisccoulson
status: Triaged → In Progress
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

Chris Coulson (chrisccoulson) wrote :

Try again!

Chris Coulson (chrisccoulson) wrote :

Here is the delta to Debian

Changed in f-spot:
status: In Progress → Triaged
Chris Coulson (chrisccoulson) wrote :

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

Chris Coulson (chrisccoulson) wrote :

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

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

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
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
Iain Lane (laney) wrote :
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  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

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