cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Bowser <bows...@gmail.com>
Subject Re: PluginManager.java regression cordova-3.2 ERROR: config.xml is missing. Add res/xml/config.xml to your project
Date Wed, 11 Dec 2013 21:35:10 GMT
Here's the log:
https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=log;h=97ad4d84ce333f05d6240b75f65da8536bcc3eb1

On Wed, Dec 11, 2013 at 1:22 PM, Axel Nennker <ignisvulpis@gmail.com> wrote:
> 3.3 will be released tomorrow, right?
>
> -Axel
>
> (Are there more patches put in after rc1?)
>
>
> 2013/12/11 Joe Bowser <bowserj@gmail.com>
>>
>> It appears that your issue has been fixed by the changes made in
>> CB-5481.  This patch was put in AFTER rc1 was released.
>> https://issues.apache.org/jira/browse/CB-5481
>>
>> On Wed, Dec 11, 2013 at 12:44 PM, Axel Nennker <ignisvulpis@gmail.com>
>> wrote:
>> > Here is the issue on JIRA:
>> > https://issues.apache.org/jira/browse/CB-5633
>> >
>> > This code works with 3.3-rc1:
>> >
>> >    public void loadPlugins() {
>> >       Activity activity = this.ctx.getActivity();
>> >       Log.d(this.getClass().getName(),
>> >
>> > "activity.getClass().getPackage().getName()="+activity.getClass().getPackage().getName());
>> >       Intent intent =
>> >
>> > activity.getPackageManager().getLaunchIntentForPackage(activity.getPackageName());
>> >       Log.d(this.getClass().getName(),
>> > "intent.getPackage()="+intent.getPackage());
>> >       int id = activity.getResources().getIdentifier("config", "xml",
>> > intent.getPackage());
>> >         if (id == 0) {
>> >
>> > -Axel
>> >
>> >
>> >
>> > 2013/12/11 Joe Bowser <bowserj@gmail.com>
>> >>
>> >> Your bug doesn't exist unless it has a JIRA issue.
>> >>
>> >> On Wed, Dec 11, 2013 at 12:25 PM, Axel Nennker <ignisvulpis@gmail.com>
>> >> wrote:
>> >> > Jesse,
>> >> > I did not create an JIRA issue yet because I do not have a small
>> >> > sample
>> >> > project to reproduce this.
>> >> > I am using the command line to create this cordova app.
>> >> > The project has a config.xml and it starts correctly (which is prove
>> >> > that it
>> >> > has the config.xml. otherwise I would see the spinner eternally).
>> >> > The version is 3.3-rc1 (Installed 30 minutes ago).
>> >> >
>> >> > I have an ant script that does the work. I would not 100%-ly rule it
>> >> > out
>> >> > that this build.xml is missing some steps. It creates the projects
>> >> > folders
>> >> > (www, merges, etc) and then calls e.g. cordova platform add android
>> >> > and
>> >> > cordova build. "cordova create hello com.example.hello HelloWorld"
is
>> >> > NOT
>> >> > called.
>> >> > I attached my build.xml and the AndroidManifest.xml
>> >> >
>> >> > -Axel
>> >> >
>> >> > config.xml is there where it belongs.
>> >> >
>> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l
>> >> > platforms/android/res/xml/
>> >> > insgesamt 4
>> >> > -rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml
>> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
>> >> >
>> >> >
>> >> >
>> >> >
>> >> >
>> >> > 2013/12/11 Jesse <purplecabbage@gmail.com>
>> >> >>
>> >> >> Is this a JIRA issue and reproducable steps yet?
>> >> >> Does the project that is failing actually have a config.xml file?
>> >> >> How are you creating the project? Platform script, cordova-cli?
What
>> >> >> version of the cli? ...
>> >> >>
>> >> >> 3.3.0 is already tagged and ready to go, so this needs reproducible
>> >> >> steps
>> >> >> to get attention.
>> >> >>
>> >> >>
>> >> >> @purplecabbage
>> >> >> risingj.com
>> >> >>
>> >> >>
>> >> >> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker
>> >> >> <ignisvulpis@gmail.com>wrote:
>> >> >>
>> >> >> > Sorry no. It does not work on 3.3-rc1
>> >> >> > Axel
>> >> >> >
>> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v
>> >> >> > 3.3.0-rc.1
>> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$
>> >> >> >
>> >> >> > same error:
>> >> >> > D/CordovaActivity( 4871): CordovaActivity.init()
>> >> >> > D/CordovaWebView( 4871): >>>
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> >> >> > D/PluginManager( 4871): init()
>> >> >> > E/PluginManager( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > =====================================================================================
>> >> >> > E/PluginManager( 4871): ERROR: config.xml is missing.  Add
>> >> >> > res/xml/config.xml to your project.
>> >> >> > E/PluginManager( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml
>> >> >> > E/PluginManager( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > =====================================================================================
>> >> >> > D/CordovaWebView( 4871): >>> loadUrlNow()
>> >> >> > D/ItemImport( 4871): onResume() loadURL started:
>> >> >> > D/CordovaActivity( 4871):
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json)
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > 2013/12/11 Andrew Grieve <agrieve@chromium.org>
>> >> >> >
>> >> >> > > If it worked in 3.1 and broke in 3.2, then it's very
likely
>> >> >> > > CB-5481<https://issues.apache.org/jira/browse/CB-5481>
and
>> >> >> > > is fixed in 3.3
>> >> >> > >
>> >> >> > >
>> >> >> > > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser <bowserj@gmail.com>
>> >> >> > > wrote:
>> >> >> > >
>> >> >> > > > No. As far as I'm concerned this problem doesn't
exist until
>> >> >> > > > it
>> >> >> > > > is a
>> >> >> > JIRA
>> >> >> > > > issue with some sample code that I can use to reproduce
the
>> >> >> > > > error. I
>> >> >> > > doubt
>> >> >> > > > that this was a regression since we never explicitly
supported
>> >> >> > > > your
>> >> >> > > > use
>> >> >> > > > case to begin with.
>> >> >> > > >
>> >> >> > > > Of course, patches are always welcome.
>> >> >> > > >  On 11 Dec 2013 07:58, "Axel Nennker" <ignisvulpis@gmail.com>
>> >> >> > > > wrote:
>> >> >> > > >
>> >> >> > > > > No. I intented this to be on the list. Gmail
UI irks.
>> >> >> > > > > Could this be fixed in 3.3?
>> >> >> > > > > I know it is late but...
>> >> >> > > > >
>> >> >> > > > > -Axel
>> >> >> > > > >
>> >> >> > > > >
>> >> >> > > > > 2013/12/11 Joe Bowser <bowserj@gmail.com>
>> >> >> > > > >
>> >> >> > > > > > Did you mean to to send this to me personally?
>> >> >> > > > > >
>> >> >> > > > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel
Nennker <
>> >> >> > ignisvulpis@gmail.com
>> >> >> > > >
>> >> >> > > > > > wrote:
>> >> >> > > > > > > Well, this repeatable. Use 3.2 fail,
go back to 3.1 win.
>> >> >> > > > > > > The config.xml is where it belongs
but PluginManager
>> >> >> > > > > > > does
>> >> >> > > > > > > not
>> >> >> > find
>> >> >> > > it
>> >> >> > > > > in
>> >> >> > > > > > a
>> >> >> > > > > > > certain (legal) setting.
>> >> >> > > > > > > When the app is started "normally"
PluginManager finds
>> >> >> > > > > > > config.xml
>> >> >> > > > > (which
>> >> >> > > > > > > proves that is at the correct location)
>> >> >> > > > > > > When the app is started because an
IntentFilter triggers
>> >> >> > > > > > > then
>> >> >> > > > > > > PluginInManager does not find it.
>> >> >> > > > > > >
>> >> >> > > > > > > The "main" activity is in one class
like a.b.c.Main
>> >> >> > > > > > > while
>> >> >> > > > > > > the
>> >> >> > > > > > IntentFilter
>> >> >> > > > > > > activity is in another e.g. u.v.w.IF
>> >> >> > > > > > > IF starts Main successfully but -
my guess is - that
>> >> >> > PluginManager
>> >> >> > > > > tries
>> >> >> > > > > > to
>> >> >> > > > > > > find config.xml in package u.v.w
>> >> >> > > > > > >
>> >> >> > > > > > > The JUNIT/spec test should be updated
to catch this
>> >> >> > > > > > > situation
>> >> >> > > > > > > if
>> >> >> > > > > > possible.
>> >> >> > > > > > > I guess I could change the package
names so that IF is
>> >> >> > > > > > > in
>> >> >> > > > > > > a.b.c
>> >> >> > too
>> >> >> > > > but
>> >> >> > > > > > it
>> >> >> > > > > > > worked in 3.1 and I see no reason
why this is "complex"
>> >> >> > > > > > > The u.v.w.IF features are implemented
in a plugin that
>> >> >> > > > > > > registers
>> >> >> > > the
>> >> >> > > > > > intent
>> >> >> > > > > > > filter in the app's AndroidManifest.
It is developed by
>> >> >> > > > > > > my
>> >> >> > > colleagues
>> >> >> > > > > who
>> >> >> > > > > > > chose the u.v.w package name. I am
using this plugin in
>> >> >> > > > > > > my
>> >> >> > > > > > > app
>> >> >> > that
>> >> >> > > > has
>> >> >> > > > > > the
>> >> >> > > > > > > a.b.c package namespace.
>> >> >> > > > > > > I do not agree that this a complex
or irregular use of
>> >> >> > > > > > > cordova
>> >> >> > > > > > > or
>> >> >> > > > > > Intents.
>> >> >> > > > > > >
>> >> >> > > > > > > Please revert the code back to the
3.1 behaviour.
>> >> >> > > > > > >
>> >> >> > > > > > > Axel
>> >> >> > > > > > >
>> >> >> > > > > > >
>> >> >> > > > > > > 2013/12/11 Joe Bowser <bowserj@gmail.com>
>> >> >> > > > > > >>
>> >> >> > > > > > >> I haven't noticed this during
any of our testing, nor
>> >> >> > > > > > >> have
>> >> >> > > > > > >> I
>> >> >> > > noticed
>> >> >> > > > > > >> this when hacking with Google
Glass, which uses the
>> >> >> > > > > > >> Google
>> >> >> > > > > > >> Glass
>> >> >> > > > voice
>> >> >> > > > > > >> trigger intent to launch Cordova,
not the LAUNCHER
>> >> >> > > > > > >> intent
>> >> >> > > > > > >> like
>> >> >> > the
>> >> >> > > > > > >> launch menu.  Furthermore, our
JUnit tests haven't
>> >> >> > > > > > >> caught
>> >> >> > > > > > >> the
>> >> >> > > tests
>> >> >> > > > > > >> either, and they depend on at
least three plugins to be
>> >> >> > > > > > >> loaded
>> >> >> > by
>> >> >> > > > > > >> PluginManager from config.xml.
 Honestly, this only
>> >> >> > > > > > >> breaks
>> >> >> > > > > > >> if
>> >> >> > you
>> >> >> > > > > > >> don't know what you're doing
with Intents.
>> >> >> > > > > > >>
>> >> >> > > > > > >> Honestly, It sounds to me like
your config.xml isn't in
>> >> >> > > > > > >> the
>> >> >> > > correct
>> >> >> > > > > > >> place in your project, especially
if you're trying to
>> >> >> > > > > > >> do
>> >> >> > something
>> >> >> > > > > > >> complex with Android and intents.
 I would check that
>> >> >> > > > > > >> making
>> >> >> > these
>> >> >> > > > > > >> sorts of statements.  If config.xml
wasn't being read,
>> >> >> > > > > > >> it
>> >> >> > > > > > >> would
>> >> >> > > have
>> >> >> > > > > > >> been caught right away, since
everything is a plugin
>> >> >> > > > > > >> now.
>> >> >> > > > > > >>
>> >> >> > > > > > >>
>> >> >> > > > > > >>
>> >> >> > > > > > >> On Wed, Dec 11, 2013 at 4:33
AM,
>> >> >> > > > > > >> <Axel.Nennker@telekom.de>
>> >> >> > > wrote:
>> >> >> > > > > > >> > Hi,
>> >> >> > > > > > >> >
>> >> >> > > > > > >> > We are seeing some strange
effect on Android when we
>> >> >> > > > > > >> > move
>> >> >> > > > > > >> > from
>> >> >> > > > > > >> > 3.1.0-0.2.0 to a newer version.
>> >> >> > > > > > >> > We tried with 3.2.0-0.4.0
and 3.3-rc1
>> >> >> > > > > > >> >
>> >> >> > > > > > >> > I think that PluginManager
changed and now fails to
>> >> >> > > > > > >> > load
>> >> >> > > > config.xml
>> >> >> > > > > > >> >
>> >> >> > > > > > >> > This happens in my project
when the app is opened by
>> >> >> > > > > > >> > the
>> >> >> > > operating
>> >> >> > > > > > >> > system because it is configured
so through an intent
>> >> >> > > > > > >> > filter.
>> >> >> > > > > > >> > The original activity is
not a child of DroidGap but
>> >> >> > > > > > >> > starts
>> >> >> > the
>> >> >> > > > > "main"
>> >> >> > > > > > >> > activity using startActivity.
>> >> >> > > > > > >> >
>> >> >> > > > > > >> > The code to get the id of
the config.xml file is not
>> >> >> > > > > > >> > working
>> >> >> > in
>> >> >> > > > this
>> >> >> > > > > > >> > case.
>> >> >> > > > > > >> >
>> >> >> > > > > > >> > Why was the 3.1.0 code changed?
>> >> >> > > > > > >> >
>> >> >> > > > > > >> > Cheers
>> >> >> > > > > > >> > Axel
>> >> >> > > > > > >> >
>> >> >> > > > > > >> > D/CordovaActivity(12320):
CordovaActivity.init()
>> >> >> > > > > > >> > D/CordovaWebView(12320):
>>>
>> >> >> > > > > > >> >
>> >> >> > > > > >
>> >> >> > > > >
>> >> >> > > >
>> >> >> > >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json)
>> >> >> > > > > > >> > D/PluginManager(12320):
init()
>> >> >> > > > > > >> > E/PluginManager(12320):
>> >> >> > > > > > >> >
>> >> >> > > > > >
>> >> >> > > > >
>> >> >> > > >
>> >> >> > >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > =====================================================================================
>> >> >> > > > > > >> > E/PluginManager(12320):
ERROR: config.xml is missing.
>> >> >> > > > > > >> > Add
>> >> >> > > > > > >> > res/xml/config.xml to your
project.
>> >> >> > > > > > >> > E/PluginManager(12320):
>> >> >> > > > > > >> >
>> >> >> > > > > >
>> >> >> > > > >
>> >> >> > > >
>> >> >> > >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml
>> >> >> > > > > > >> > E/PluginManager(12320):
>> >> >> > > > > > >> >
>> >> >> > > > > >
>> >> >> > > > >
>> >> >> > > >
>> >> >> > >
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > =====================================================================================
>> >> >> > > > > > >> > D/CordovaWebView(12320):
>>> loadUrlNow()
>> >> >> > > > > > >> > D/ItemImport(12320): onResume()
loadURL started:
>> >> >> > > > > > >
>> >> >> > > > > > >
>> >> >> > > > > >
>> >> >> > > > >
>> >> >> > > >
>> >> >> > >
>> >> >> >
>> >> >
>> >> >
>> >
>> >
>
>

Mime
View raw message