Swiftsuspenders package structure

fmotagarcia's Avatar

fmotagarcia

Aug 29, 2013 @ 11:25 AM

Hi,

At my company we developed some games using RL1. They were originally targeting web, but were then converted for mobile. All games are inside the same app.

For next games we will start using RL2. With mobile I now have a problem with Swiftsuspenders. Since all games are in the same applicationDomain SS1 and SS2 conflict (there is no problem with RL, since the package is different now). The way I'm working around this is branching SS1 and changing it's package structure.

My question is, could you guys change the package structure in SS2 so that RL1 and RL2 apps can coexist in the same appDomain?

  1. Support Staff 1 Posted by Ondina D.F. on Aug 30, 2013 @ 07:51 AM

    Ondina D.F.'s Avatar

    Hello,

    You're right, there should be a compatibility RL1 branch.

    I opened an issue:
    https://github.com/robotlegs/robotlegs-framework/issues/153

    Ondina

  2. 2 Posted by fmotagarcia on Aug 30, 2013 @ 10:54 AM

    fmotagarcia's Avatar

    wouldn't it be quicker and easier to change swiftsuspenders? I think non backward compatible versions of any library should use a different namespace (like robotlegs bender)

  3. Support Staff 3 Posted by Ondina D.F. on Aug 30, 2013 @ 11:20 AM

    Ondina D.F.'s Avatar

    Robotlegs v.1 adapters (package org.robotlegs.adapters - SwiftSuspendersReflector and SwiftSuspendersInjector) have to reflect the changes to Swiftsuspenders' package name.

  4. 4 Posted by fmotagarcia on Aug 30, 2013 @ 11:39 AM

    fmotagarcia's Avatar

    What I mean is change Swiftsuspenders 2 and RL 2, leaving RL1 the way it is now. Seems like it makes more sense, as it'd require no changes in legacy code

  5. Ondina D.F. closed this discussion on Oct 11, 2013 @ 08:52 AM.

Comments are currently closed for this discussion. You can start a new one.

Keyboard shortcuts

Generic

? Show this help
ESC Blurs the current field

Comment Form

r Focus the comment reply box
^ + ↩ Submit the comment

You can use Command ⌘ instead of Control ^ on Mac