Bug 169170 - [Cocoa] libwebrtc.dylib’s current version is fixed at 1.0.0
Summary: [Cocoa] libwebrtc.dylib’s current version is fixed at 1.0.0
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: mitz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-04 10:12 PST by mitz
Modified: 2017-03-04 23:41 PST (History)
3 users (show)

See Also:


Attachments
Set DYLIB_CURRENT_VERSION (4.48 KB, patch)
2017-03-04 10:21 PST, mitz
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2017-03-04 10:12:02 PST
libwebrtc.dylib’s current version is fixed at 1.0.0, so it’s impossible to tell which version of the library was present at link time, and harder to diagnose version incompatibilities at runtime.

Patch forthcoming.
Comment 1 mitz 2017-03-04 10:21:33 PST
Created attachment 303398 [details]
Set DYLIB_CURRENT_VERSION
Comment 2 Alex Christensen 2017-03-04 23:00:43 PST
Comment on attachment 303398 [details]
Set DYLIB_CURRENT_VERSION

View in context: https://bugs.webkit.org/attachment.cgi?id=303398&action=review

> Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig:1
> +// Copyright (C) 2009 Apple Inc. All rights reserved.

2017?

> Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig:36
> +SYSTEM_VERSION_PREFIX[sdk=iphone*] = 8;

Why 8?  This definitely matches what is done elsewhere.
Comment 3 mitz 2017-03-04 23:12:55 PST
Comment on attachment 303398 [details]
Set DYLIB_CURRENT_VERSION

View in context: https://bugs.webkit.org/attachment.cgi?id=303398&action=review

>> Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig:1
>> +// Copyright (C) 2009 Apple Inc. All rights reserved.
> 
> 2017?

Making a copy of this file in 2017 doesn’t introduce any new content.

>> Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig:36
>> +SYSTEM_VERSION_PREFIX[sdk=iphone*] = 8;
> 
> Why 8?  This definitely matches what is done elsewhere.

My recollection is during the time the iOS fork of WebKit existed, it would occasionally update this to match the most recent upstream Mac OS X version, and that most recently happened with 10.8. Because of the way iOS software updates are installed, it’s not important to use different prefixes for different target OS versions. Because the libwebrtc project doesn’t build any bundles, it doesn’t use the bundle version prefix, but we use the same Version.xcconfig across all projects.
Comment 4 WebKit Commit Bot 2017-03-04 23:41:28 PST
Comment on attachment 303398 [details]
Set DYLIB_CURRENT_VERSION

Clearing flags on attachment: 303398

Committed r213432: <http://trac.webkit.org/changeset/213432>
Comment 5 WebKit Commit Bot 2017-03-04 23:41:33 PST
All reviewed patches have been landed.  Closing bug.