You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by Dmitry Blotsky <db...@microsoft.com> on 2015/07/17 23:13:57 UTC
[CI] iOS build breaking
https://ci.apache.org/builders/cordova-ios/builds/36/steps/running-tests/logs/stdio
CDVCamera.m:23:9: fatal error: 'Cordova/NSData+Base64.h' file not found
#import <Cordova/NSData+Base64.h>
^
1 error generated.
Kindly,
Dmitry
Re: [CI] iOS build breaking
Posted by Dmitry Blotsky <db...@microsoft.com>.
That’s good to hear; then it’s not too bad. Any ideas on CB-8993? This seems like it was fixed, but now is breaking again.
Kindly,
Dmitry
> On Jul 21, 2015, at 4:12 PM, Shazron <sh...@gmail.com> wrote:
>
> This is not true -- the v1.2.0 (latest) version of the camera plugin builds
> with cordova-ios 3.8.0 (released), it just won't build with cordova-ios
> master (by the time cordova-ios 4.0 is released, a new plugin release will
> be out with the required changes).
>
> The master version of the camera plugin builds with both cordova-ios 3.8.0
> and cordova-ios master.
>
> I just tested and verified these scenarios.
>
>
> On Tue, Jul 21, 2015 at 10:55 AM, Nikhil Khandelwal <ni...@microsoft.com>
> wrote:
>
>> There is another JIRA that is needed, that the latest version of the
>> 'camera' plugin cannot work against released version of iOS. If that's by
>> design (which is breaking and unfortunate) - the plugin.xml needs to
>> specify that.
>>
>> Thanks,
>> Nikhil
>>
>>
>> -----Original Message-----
>> From: Dmitry Blotsky [mailto:dblotsky@microsoft.com]
>> Sent: Tuesday, July 21, 2015 1:58 AM
>> To: dev@cordova.apache.org
>> Subject: Re: [CI] iOS build breaking
>>
>> It seems that Mobilespec is using NPM plugins instead of the local ones:
>> https://ci.apache.org/builders/cordova-ios/builds/50/steps/creating-mobilespec-app/logs/stdio.
>> And this is because ‘cordova plugin add’ is not respecting the
>> ‘--searchpath’ argument. Bug reopened: CB-8993.
>>
>> Any idea how this happened? Also, whoever fixes this should add tests for
>> it.
>>
>> Kindly,
>> Dmitry
>>
>> On Jul 18, 2015, at 1:27 AM, Shazron <shazron@gmail.com<mailto:
>> shazron@gmail.com>> wrote:
>>
>> Not sure what's going on here that line does not exist in master:
>>
>> https://github.com/apache/cordova-plugin-camera/blob/master/src/ios/CDVCamera.m#L23
>>
>> I fixed this bug 23 days ago:
>>
>> https://github.com/apache/cordova-plugin-camera/commit/63110ea54c16d1e0958f0986b3d5a43b254df13a
>>
>> Perhaps the CI is getting the released version (r1.2.0) instead of master?
>>
>>
>>
>> On Fri, Jul 17, 2015 at 2:13 PM, Dmitry Blotsky <db...@microsoft.com>
>> wrote:
>>
>>
>>
>> https://ci.apache.org/builders/cordova-ios/builds/36/steps/running-tests/logs/stdio
>>
>>
>> CDVCamera.m:23:9: fatal error: 'Cordova/NSData+Base64.h' file not found
>> #import <Cordova/NSData+Base64.h>
>> ^
>> 1 error generated.
>>
>>
>> Kindly,
>> Dmitry
>>
>>
>>
Re: [CI] iOS build breaking
Posted by Shazron <sh...@gmail.com>.
This is not true -- the v1.2.0 (latest) version of the camera plugin builds
with cordova-ios 3.8.0 (released), it just won't build with cordova-ios
master (by the time cordova-ios 4.0 is released, a new plugin release will
be out with the required changes).
The master version of the camera plugin builds with both cordova-ios 3.8.0
and cordova-ios master.
I just tested and verified these scenarios.
On Tue, Jul 21, 2015 at 10:55 AM, Nikhil Khandelwal <ni...@microsoft.com>
wrote:
> There is another JIRA that is needed, that the latest version of the
> 'camera' plugin cannot work against released version of iOS. If that's by
> design (which is breaking and unfortunate) - the plugin.xml needs to
> specify that.
>
> Thanks,
> Nikhil
>
>
> -----Original Message-----
> From: Dmitry Blotsky [mailto:dblotsky@microsoft.com]
> Sent: Tuesday, July 21, 2015 1:58 AM
> To: dev@cordova.apache.org
> Subject: Re: [CI] iOS build breaking
>
> It seems that Mobilespec is using NPM plugins instead of the local ones:
> https://ci.apache.org/builders/cordova-ios/builds/50/steps/creating-mobilespec-app/logs/stdio.
> And this is because ‘cordova plugin add’ is not respecting the
> ‘--searchpath’ argument. Bug reopened: CB-8993.
>
> Any idea how this happened? Also, whoever fixes this should add tests for
> it.
>
> Kindly,
> Dmitry
>
> On Jul 18, 2015, at 1:27 AM, Shazron <shazron@gmail.com<mailto:
> shazron@gmail.com>> wrote:
>
> Not sure what's going on here that line does not exist in master:
>
> https://github.com/apache/cordova-plugin-camera/blob/master/src/ios/CDVCamera.m#L23
>
> I fixed this bug 23 days ago:
>
> https://github.com/apache/cordova-plugin-camera/commit/63110ea54c16d1e0958f0986b3d5a43b254df13a
>
> Perhaps the CI is getting the released version (r1.2.0) instead of master?
>
>
>
> On Fri, Jul 17, 2015 at 2:13 PM, Dmitry Blotsky <db...@microsoft.com>
> wrote:
>
>
>
> https://ci.apache.org/builders/cordova-ios/builds/36/steps/running-tests/logs/stdio
>
>
> CDVCamera.m:23:9: fatal error: 'Cordova/NSData+Base64.h' file not found
> #import <Cordova/NSData+Base64.h>
> ^
> 1 error generated.
>
>
> Kindly,
> Dmitry
>
>
>
RE: [CI] iOS build breaking
Posted by Nikhil Khandelwal <ni...@microsoft.com>.
There is another JIRA that is needed, that the latest version of the 'camera' plugin cannot work against released version of iOS. If that's by design (which is breaking and unfortunate) - the plugin.xml needs to specify that.
Thanks,
Nikhil
-----Original Message-----
From: Dmitry Blotsky [mailto:dblotsky@microsoft.com]
Sent: Tuesday, July 21, 2015 1:58 AM
To: dev@cordova.apache.org
Subject: Re: [CI] iOS build breaking
It seems that Mobilespec is using NPM plugins instead of the local ones: https://ci.apache.org/builders/cordova-ios/builds/50/steps/creating-mobilespec-app/logs/stdio. And this is because ‘cordova plugin add’ is not respecting the ‘--searchpath’ argument. Bug reopened: CB-8993.
Any idea how this happened? Also, whoever fixes this should add tests for it.
Kindly,
Dmitry
On Jul 18, 2015, at 1:27 AM, Shazron <sh...@gmail.com>> wrote:
Not sure what's going on here that line does not exist in master:
https://github.com/apache/cordova-plugin-camera/blob/master/src/ios/CDVCamera.m#L23
I fixed this bug 23 days ago:
https://github.com/apache/cordova-plugin-camera/commit/63110ea54c16d1e0958f0986b3d5a43b254df13a
Perhaps the CI is getting the released version (r1.2.0) instead of master?
On Fri, Jul 17, 2015 at 2:13 PM, Dmitry Blotsky <db...@microsoft.com>
wrote:
https://ci.apache.org/builders/cordova-ios/builds/36/steps/running-tests/logs/stdio
CDVCamera.m:23:9: fatal error: 'Cordova/NSData+Base64.h' file not found #import <Cordova/NSData+Base64.h>
^
1 error generated.
Kindly,
Dmitry
Re: [CI] iOS build breaking
Posted by Dmitry Blotsky <db...@microsoft.com>.
It seems that Mobilespec is using NPM plugins instead of the local ones: https://ci.apache.org/builders/cordova-ios/builds/50/steps/creating-mobilespec-app/logs/stdio. And this is because ‘cordova plugin add’ is not respecting the ‘--searchpath’ argument. Bug reopened: CB-8993.
Any idea how this happened? Also, whoever fixes this should add tests for it.
Kindly,
Dmitry
On Jul 18, 2015, at 1:27 AM, Shazron <sh...@gmail.com>> wrote:
Not sure what's going on here that line does not exist in master:
https://github.com/apache/cordova-plugin-camera/blob/master/src/ios/CDVCamera.m#L23
I fixed this bug 23 days ago:
https://github.com/apache/cordova-plugin-camera/commit/63110ea54c16d1e0958f0986b3d5a43b254df13a
Perhaps the CI is getting the released version (r1.2.0) instead of master?
On Fri, Jul 17, 2015 at 2:13 PM, Dmitry Blotsky <db...@microsoft.com>
wrote:
https://ci.apache.org/builders/cordova-ios/builds/36/steps/running-tests/logs/stdio
CDVCamera.m:23:9: fatal error: 'Cordova/NSData+Base64.h' file not found
#import <Cordova/NSData+Base64.h>
^
1 error generated.
Kindly,
Dmitry
Re: [CI] iOS build breaking
Posted by Shazron <sh...@gmail.com>.
Not sure what's going on here that line does not exist in master:
https://github.com/apache/cordova-plugin-camera/blob/master/src/ios/CDVCamera.m#L23
I fixed this bug 23 days ago:
https://github.com/apache/cordova-plugin-camera/commit/63110ea54c16d1e0958f0986b3d5a43b254df13a
Perhaps the CI is getting the released version (r1.2.0) instead of master?
On Fri, Jul 17, 2015 at 2:13 PM, Dmitry Blotsky <db...@microsoft.com>
wrote:
>
> https://ci.apache.org/builders/cordova-ios/builds/36/steps/running-tests/logs/stdio
>
>
> CDVCamera.m:23:9: fatal error: 'Cordova/NSData+Base64.h' file not found
> #import <Cordova/NSData+Base64.h>
> ^
> 1 error generated.
>
>
> Kindly,
> Dmitry
>