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 795312 - Drop support for old versions of libmtp < 1.1.12
Drop support for old versions of libmtp < 1.1.12
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: mtp backend
git master
Other Linux
: Normal normal
: ---
Assigned To: Philip Langdale
gvfs-maint
Depends on:
Blocks:
 
 
Reported: 2018-04-17 02:47 UTC by Philip Langdale
Modified: 2018-04-18 03:39 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
Remove conditionals for libmtp < 1.1.12 (9.68 KB, patch)
2018-04-17 03:07 UTC, Philip Langdale
accepted-commit_now Details | Review

Description Philip Langdale 2018-04-17 02:47:35 UTC
We've kept all of the conditionals we've accumulated over the years as we added support for new libmtp functionality. It's become a pretty long list of versions, and there's no reason to support all the ancient releases. So, let's drop the supported versions older than 1.1.12. The latest 1.1.15 release has only been out for a few weeks, so it's a bit step to require that yet. This will bring us down to just two versions.
Comment 1 Philip Langdale 2018-04-17 03:07:45 UTC
Created attachment 371006 [details] [review]
Remove conditionals for libmtp < 1.1.12
Comment 2 Ondrej Holy 2018-04-17 08:07:07 UTC
Review of attachment 371006 [details] [review]:

Looks ok, thanks!

::: meson.build
@@ +439,3 @@
 
+  libmtp_dep = dependency('libmtp', version: '>= 1.1.12')
+  foreach version: ['1.1.15']

nitpick: Doesn't make much sense to have foreach here now, but if you expect we will use again soon, it is probably ok...
Comment 3 Philip Langdale 2018-04-18 03:38:33 UTC
(In reply to Ondrej Holy from comment #2)
> +  libmtp_dep = dependency('libmtp', version: '>= 1.1.12')
> +  foreach version: ['1.1.15']
> 
> nitpick: Doesn't make much sense to have foreach here now, but if you expect
> we will use again soon, it is probably ok...

Yeah, I thought about it but I figured there's a good chance we'll introduce a new version check - and if not, then this will all just go away when we move to only supporting >= 1.1.15