Re: [OpenBabel-Devel] Appveyor tests

2017-06-09 Thread Maciek Wójcikowski 
Apparently my last name killed conda build...
https://ci.appveyor.com/project/mwojcikowski/conda-openbabel/build/1.0.23/job/hvv95j1fagio4ike


Pozdrawiam,  |  Best regards,
Maciek Wójcikowski
mac...@wojcikowski.pl

2017-06-09 15:19 GMT+02:00 Maciek Wójcikowski :

> I'm on it as we speak.
>
> 
> Pozdrawiam,  |  Best regards,
> Maciek Wójcikowski
> mac...@wojcikowski.pl
>
> 2017-06-09 15:09 GMT+02:00 Noel O'Boyle :
>
>> Thanks for the super quick response. I do appreciate it.
>>
>> There is a std::isnan via _isnan build problem which will require a
>> #define for particular MSVC versions, but that's nothing to do with my
>> pull request.
>>
>> Ok - I'm going to do ahead and silence the tests
>>
>> - Noel
>>
>> On 9 June 2017 at 13:56, Maciek Wójcikowski 
>> wrote:
>> > I'm getting different errors on current master, see
>> > https://ci.appveyor.com/project/mwojcikowski/conda-openbabel
>> /build/1.0.19
>> > I need to fix conda recipe for git master first, but I think we should
>> merge
>> > it as is and figure out what is the issue with tests later on.
>> >
>> > 
>> > Pozdrawiam,  |  Best regards,
>> > Maciek Wójcikowski
>> > mac...@wojcikowski.pl
>> >
>> > 2017-06-09 14:23 GMT+02:00 Noel O'Boyle :
>> >>
>> >> I'm keen to get my code merged, so I'm going to press you on this.
>> >> I've spent several months getting these tests to 100% passing. I'm
>> >> happy to send a screenshot showing this on my Windows machine.
>> >>
>> >> - Noel
>> >>
>> >> On 6 June 2017 at 20:52, Noel O'Boyle  wrote:
>> >> > The link is on the PR:
>> >> > https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.501
>> >> >
>> >> > Ok - I'll wait until you confirm.
>> >> >
>> >> > - Noel
>> >> >
>> >> > On 5 June 2017 at 19:18, Maciek Wójcikowski 
>> >> > wrote:
>> >> >> We need to verify that it doesn't mess up the conda packages for
>> >> >> windows.
>> >> >> I'll have some time by the end of the week.
>> >> >>
>> >> >> PS.
>> >> >> https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.481
>> this
>> >> >> build says 100% tests passed. Am I looking at the wrong one?
>> >> >>
>> >> >> 
>> >> >> Pozdrawiam,  |  Best regards,
>> >> >> Maciek Wójcikowski
>> >> >> mac...@wojcikowski.pl
>> >> >>
>> >> >> 2017-06-05 19:17 GMT+02:00 Noel O'Boyle :
>> >> >>>
>> >> >>> Hey Geoff,
>> >> >>>
>> >> >>> I'm going to turn off tests for the Appveyor build due to the
>> problems
>> >> >>> with PR#1572. I don't think they are real - do you remember a
>> similiar
>> >> >>> issue with a PR of David Koes, where I had exactly the same problem
>> >> >>> (except here it's multiple tests)? Something's funny with the VMs
>> that
>> >> >>> Appveyor use. If it's real, obviously we'll have to sort it out,
>> but
>> >> >>> the tests pass for me. The main reason for the windows build is
>> simply
>> >> >>> for compilation, so while not ideal, I don't think it's a major
>> issue.
>> >> >>>
>> >> >>> - Noel
>> >> >>>
>> >> >>>
>> >> >>>
>> >> >>> 
>> --
>> >> >>> Check out the vibrant tech community on one of the world's most
>> >> >>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>> >> >>> ___
>> >> >>> OpenBabel-Devel mailing list
>> >> >>> OpenBabel-Devel@lists.sourceforge.net
>> >> >>> https://lists.sourceforge.net/lists/listinfo/openbabel-devel
>> >> >>
>> >> >>
>> >
>> >
>>
>
>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel

Re: [OpenBabel-Devel] Appveyor tests

2017-06-09 Thread Maciek Wójcikowski 
I'm on it as we speak.


Pozdrawiam,  |  Best regards,
Maciek Wójcikowski
mac...@wojcikowski.pl

2017-06-09 15:09 GMT+02:00 Noel O'Boyle :

> Thanks for the super quick response. I do appreciate it.
>
> There is a std::isnan via _isnan build problem which will require a
> #define for particular MSVC versions, but that's nothing to do with my
> pull request.
>
> Ok - I'm going to do ahead and silence the tests
>
> - Noel
>
> On 9 June 2017 at 13:56, Maciek Wójcikowski  wrote:
> > I'm getting different errors on current master, see
> > https://ci.appveyor.com/project/mwojcikowski/conda-
> openbabel/build/1.0.19
> > I need to fix conda recipe for git master first, but I think we should
> merge
> > it as is and figure out what is the issue with tests later on.
> >
> > 
> > Pozdrawiam,  |  Best regards,
> > Maciek Wójcikowski
> > mac...@wojcikowski.pl
> >
> > 2017-06-09 14:23 GMT+02:00 Noel O'Boyle :
> >>
> >> I'm keen to get my code merged, so I'm going to press you on this.
> >> I've spent several months getting these tests to 100% passing. I'm
> >> happy to send a screenshot showing this on my Windows machine.
> >>
> >> - Noel
> >>
> >> On 6 June 2017 at 20:52, Noel O'Boyle  wrote:
> >> > The link is on the PR:
> >> > https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.501
> >> >
> >> > Ok - I'll wait until you confirm.
> >> >
> >> > - Noel
> >> >
> >> > On 5 June 2017 at 19:18, Maciek Wójcikowski 
> >> > wrote:
> >> >> We need to verify that it doesn't mess up the conda packages for
> >> >> windows.
> >> >> I'll have some time by the end of the week.
> >> >>
> >> >> PS.
> >> >> https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.481
> this
> >> >> build says 100% tests passed. Am I looking at the wrong one?
> >> >>
> >> >> 
> >> >> Pozdrawiam,  |  Best regards,
> >> >> Maciek Wójcikowski
> >> >> mac...@wojcikowski.pl
> >> >>
> >> >> 2017-06-05 19:17 GMT+02:00 Noel O'Boyle :
> >> >>>
> >> >>> Hey Geoff,
> >> >>>
> >> >>> I'm going to turn off tests for the Appveyor build due to the
> problems
> >> >>> with PR#1572. I don't think they are real - do you remember a
> similiar
> >> >>> issue with a PR of David Koes, where I had exactly the same problem
> >> >>> (except here it's multiple tests)? Something's funny with the VMs
> that
> >> >>> Appveyor use. If it's real, obviously we'll have to sort it out, but
> >> >>> the tests pass for me. The main reason for the windows build is
> simply
> >> >>> for compilation, so while not ideal, I don't think it's a major
> issue.
> >> >>>
> >> >>> - Noel
> >> >>>
> >> >>>
> >> >>>
> >> >>> 
> --
> >> >>> Check out the vibrant tech community on one of the world's most
> >> >>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> >> >>> ___
> >> >>> OpenBabel-Devel mailing list
> >> >>> OpenBabel-Devel@lists.sourceforge.net
> >> >>> https://lists.sourceforge.net/lists/listinfo/openbabel-devel
> >> >>
> >> >>
> >
> >
>
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel

Re: [OpenBabel-Devel] Appveyor tests

2017-06-09 Thread Noel O'Boyle 
Thanks for the super quick response. I do appreciate it.

There is a std::isnan via _isnan build problem which will require a
#define for particular MSVC versions, but that's nothing to do with my
pull request.

Ok - I'm going to do ahead and silence the tests

- Noel

On 9 June 2017 at 13:56, Maciek Wójcikowski  wrote:
> I'm getting different errors on current master, see
> https://ci.appveyor.com/project/mwojcikowski/conda-openbabel/build/1.0.19
> I need to fix conda recipe for git master first, but I think we should merge
> it as is and figure out what is the issue with tests later on.
>
> 
> Pozdrawiam,  |  Best regards,
> Maciek Wójcikowski
> mac...@wojcikowski.pl
>
> 2017-06-09 14:23 GMT+02:00 Noel O'Boyle :
>>
>> I'm keen to get my code merged, so I'm going to press you on this.
>> I've spent several months getting these tests to 100% passing. I'm
>> happy to send a screenshot showing this on my Windows machine.
>>
>> - Noel
>>
>> On 6 June 2017 at 20:52, Noel O'Boyle  wrote:
>> > The link is on the PR:
>> > https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.501
>> >
>> > Ok - I'll wait until you confirm.
>> >
>> > - Noel
>> >
>> > On 5 June 2017 at 19:18, Maciek Wójcikowski 
>> > wrote:
>> >> We need to verify that it doesn't mess up the conda packages for
>> >> windows.
>> >> I'll have some time by the end of the week.
>> >>
>> >> PS.
>> >> https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.481 this
>> >> build says 100% tests passed. Am I looking at the wrong one?
>> >>
>> >> 
>> >> Pozdrawiam,  |  Best regards,
>> >> Maciek Wójcikowski
>> >> mac...@wojcikowski.pl
>> >>
>> >> 2017-06-05 19:17 GMT+02:00 Noel O'Boyle :
>> >>>
>> >>> Hey Geoff,
>> >>>
>> >>> I'm going to turn off tests for the Appveyor build due to the problems
>> >>> with PR#1572. I don't think they are real - do you remember a similiar
>> >>> issue with a PR of David Koes, where I had exactly the same problem
>> >>> (except here it's multiple tests)? Something's funny with the VMs that
>> >>> Appveyor use. If it's real, obviously we'll have to sort it out, but
>> >>> the tests pass for me. The main reason for the windows build is simply
>> >>> for compilation, so while not ideal, I don't think it's a major issue.
>> >>>
>> >>> - Noel
>> >>>
>> >>>
>> >>>
>> >>> --
>> >>> Check out the vibrant tech community on one of the world's most
>> >>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>> >>> ___
>> >>> OpenBabel-Devel mailing list
>> >>> OpenBabel-Devel@lists.sourceforge.net
>> >>> https://lists.sourceforge.net/lists/listinfo/openbabel-devel
>> >>
>> >>
>
>

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel

Re: [OpenBabel-Devel] Appveyor tests

2017-06-09 Thread Noel O'Boyle 
I'm keen to get my code merged, so I'm going to press you on this.
I've spent several months getting these tests to 100% passing. I'm
happy to send a screenshot showing this on my Windows machine.

- Noel

On 6 June 2017 at 20:52, Noel O'Boyle  wrote:
> The link is on the PR:
> https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.501
>
> Ok - I'll wait until you confirm.
>
> - Noel
>
> On 5 June 2017 at 19:18, Maciek Wójcikowski  wrote:
>> We need to verify that it doesn't mess up the conda packages for windows.
>> I'll have some time by the end of the week.
>>
>> PS.
>> https://ci.appveyor.com/project/baoilleach/openbabel/build/1.0.481 this
>> build says 100% tests passed. Am I looking at the wrong one?
>>
>> 
>> Pozdrawiam,  |  Best regards,
>> Maciek Wójcikowski
>> mac...@wojcikowski.pl
>>
>> 2017-06-05 19:17 GMT+02:00 Noel O'Boyle :
>>>
>>> Hey Geoff,
>>>
>>> I'm going to turn off tests for the Appveyor build due to the problems
>>> with PR#1572. I don't think they are real - do you remember a similiar
>>> issue with a PR of David Koes, where I had exactly the same problem
>>> (except here it's multiple tests)? Something's funny with the VMs that
>>> Appveyor use. If it's real, obviously we'll have to sort it out, but
>>> the tests pass for me. The main reason for the windows build is simply
>>> for compilation, so while not ideal, I don't think it's a major issue.
>>>
>>> - Noel
>>>
>>>
>>> --
>>> Check out the vibrant tech community on one of the world's most
>>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>>> ___
>>> OpenBabel-Devel mailing list
>>> OpenBabel-Devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/openbabel-devel
>>
>>

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel