After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 620842 - [GtkBuilder transition] last_import_rolls_filter
[GtkBuilder transition] last_import_rolls_filter
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: General
GIT
Other Linux
: Normal enhancement
: 0.7.1
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks: f-spot-glade
 
 
Reported: 2010-06-07 14:08 UTC by Ruben Vermeersch
Modified: 2010-06-25 08:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Migrates LastImportRollFilterDialog to GtkBuilder (19.60 KB, patch)
2010-06-21 22:22 UTC, Peter Goetz
needs-work Details | Review
Added LastImportRollFilterDialog.ui to POTFILES.in (908 bytes, patch)
2010-06-22 23:20 UTC, Peter Goetz
needs-work Details | Review
Migrates LastImportRollFilterDialog to GtkBuilder (19.61 KB, patch)
2010-06-24 21:00 UTC, Peter Goetz
committed Details | Review
Added LastImportRollFilterDialog.ui to POTFILES.in (908 bytes, patch)
2010-06-24 21:00 UTC, Peter Goetz
committed Details | Review

Description Ruben Vermeersch 2010-06-07 14:08:01 UTC
The last_import_rolls_filter needs to be migrated to GtkBuilder.
Comment 1 Ruben Vermeersch 2010-06-07 15:21:42 UTC
Instructions on how to fix this can be found here:
http://f-spot.org/GtkBuilder_Transition

Please indicate below if you are working on this (and if you stopped working on it), so we can avoid duplicate work. If you are stuck, just ask for help on IRC or the mailing list.
Comment 2 Peter Goetz 2010-06-21 22:22:28 UTC
Created attachment 164254 [details] [review]
Migrates LastImportRollFilterDialog to GtkBuilder
Comment 3 Maxxer 2010-06-22 07:57:52 UTC
Review of attachment 164254 [details] [review]:

potfiles.in please! :)
Comment 4 Peter Goetz 2010-06-22 23:20:50 UTC
Created attachment 164367 [details] [review]
Added LastImportRollFilterDialog.ui to POTFILES.in

Thanks for your review, Maxxer :). Didn't know of that file before. And as it turned out, you adjusted the potfiles.in for my CreateTagDialog-patch. So how could I have known of it ;) Now I know.

Is it correct that the potfiles.in still lists those files as glade files?
Comment 5 Peter Goetz 2010-06-22 23:58:44 UTC
It seems like that first patch doesn't work anymore, since other patches already landed in master. I must first find out how I can fix this the easiest way with git and then I'll provide new patches.
Comment 6 Maxxer 2010-06-24 07:56:06 UTC
(In reply to comment #4)
> Thanks for your review, Maxxer :). Didn't know of that file before. And as it

I thought I told you in another bug, but maybe wasn't you then! 
thanks for the update.

you will probably find info on how to update your patch at 
https://bugzilla.gnome.org/show_bug.cgi?id=620838
Comment 7 Ruben Vermeersch 2010-06-24 09:01:36 UTC
(In reply to comment #4)
> Is it correct that the potfiles.in still lists those files as glade files?

Yes, it's mainly used to specify how intltool will parse them. The format of Glade and GtkBuilder isn't that different, so the same parser can be used.
Comment 8 Ruben Vermeersch 2010-06-24 09:02:05 UTC
Comment on attachment 164367 [details] [review]
Added LastImportRollFilterDialog.ui to POTFILES.in

Please rebase this together with the other patch :-)
Comment 9 Peter Goetz 2010-06-24 21:00:12 UTC
Created attachment 164549 [details] [review]
Migrates LastImportRollFilterDialog to GtkBuilder
Comment 10 Peter Goetz 2010-06-24 21:00:54 UTC
Created attachment 164550 [details] [review]
Added LastImportRollFilterDialog.ui to POTFILES.in
Comment 11 Peter Goetz 2010-06-25 08:25:36 UTC
I rebased both patches. Now it should be possible to apply them.
Comment 12 Ruben Vermeersch 2010-06-25 08:34:31 UTC
Attachment 164550 [details] pushed as e5337f1 - Added LastImportRollFilterDialog.ui to POTFILES.in


Great work, both patches (which are clean and good) applied cleanly and I've
pushed them to master. Thanks a lot!