Closed Bug 1223400 Opened 9 years ago Closed 8 years ago

Replace SwiftKeychainWrapper source with Carthage import

Categories

(Firefox for iOS :: Build & Test, defect)

All
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fxios 2.0+ ---

People

(Reporter: sleroux, Assigned: fluffyemily)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

48 bytes, text/x-github-pull-request
sleroux
: review+
Details | Review
There's an open issue to support Carthage and it looks like it's working but might need to be updated to Swift 2.1 to avoid warnings:

https://github.com/jrendel/SwiftKeychainWrapper/issues/14

Since 2.0 -> 2.1 isn't too bad, we can even submit a patch to get it in a non-warning state. Looks like it's working though.
Blocks: 1223403
There is an open pull request that removes the warnings (see [1]). However, these are only warnings and on top they only affect the unit tests so there should be nothing blocking the build.

[1] https://github.com/jrendel/SwiftKeychainWrapper/pull/25
The PR was merged so there should be no warnings anymore.
tracking-fxios: ? → ---
Hardware: Other → All
Attached file Pull request
Part 1 of using carthage correctly - move GDCWebserver in ThirdParty to reference version fetched by carthage
Attachment #8704142 - Flags: review?(sleroux)
Assignee: nobody → etoop
Status: NEW → ASSIGNED
:sleroux - pinned GCDWebServer and SwiftKeychainWrapper to a particular version
Attachment #8704142 - Flags: review?(sleroux) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: