Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of fil@adobe.com designates
 64.18.1.181 as permitted sender)
From: Filip Maj <fil@adobe.com>
To: "callback-dev@incubator.apache.org" <callback-dev@incubator.apache.org>
CC: "jrburke@gmail.com" <jrburke@gmail.com>
Date: Wed, 22 Feb 2012 10:51:03 -0800
Subject: Re: using existing AMD implementations with cordova.js
Thread-Topic: using existing AMD implementations with cordova.js
Thread-Index: Aczxku4mYXYOQ7MLRgu4ufTvLUDEOw==
Message-ID: <CB6A78BB.571A%fil@adobe.com>
In-Reply-To: 
 <CADVgkyO0etwW4bP2NyS64taN3J_j84QBF1aBU9NHHnyMfDW5GQ@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-MacOutlook/14.14.0.111121
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Let's be very careful in recommending anything to our users as our
bikeshed require implementation is no where close to what most people
consider is "require.js".

Also we should definitely document what is/isn't supported by our require.
No async loading, for example.

I'll create a JIRA ticket to document that stuff in the README - anywhere
else that documentation should exist you guys think?

On 12-02-22 10:46 AM, "Brian LeRoux" <b@brian.io> wrote:

>> - our require.js adds a "define.remove()" function which no one else
>> implements.  I'm happy with that, but we should probably make the tests
>> sensitive to it - only do the "remove" tests if "define.remove()" is
>>present
>
>+1
>
>> - issues w/almond and requirejs not throwing exceptions when modules are
>> multiply defined or missing for require().  I've noted this issue with
>> almond before, and there's an open bug at almond for it.  Seems like
>> requirejs is also not doing this.  For this reason alone, I'd recommend
>> folks not use almond or requirejs as difficult-to-diagnose failures will
>> occur if, for some reason, a required module is absent.
>
>
>So we *can* support alternate loaders but we'd recommend just using
>https://github.com/callback/callback-js/blob/master/lib/require.js ??