netcdf entry for autoproj

Hi,

Recently I started using the netcdf reporter, I noticed that it was once
disable due to it not compiling, this seems not to be the case anymore.
Therefor I enabled it again by default, added a system dependency on netcdf
in OCL's manifest. The attached patch for autoproj provides an entry for
the netcdf packages for ubuntu and debian.

I'm sending the patch since I don't know whether to push it to autoproj
master or somewhere else.

Ruben

AttachmentSize
0001-add-netcdf-entry.patch659 bytes

netcdf entry for autoproj

2012/11/22 Ruben Smits <ruben [dot] smits [..] ...>

> Hi,

Hi Ruben,

>
> Recently I started using the netcdf reporter, I noticed that it was once
> disable due to it not compiling, this seems not to be the case anymore.
> Therefor I enabled it again by default, added a system dependency on netcdf
> in OCL's manifest. The attached patch for autoproj provides an entry for
> the netcdf packages for ubuntu and debian.
>

You should find a way to make this dependency optional: I think most of the
people using OCL don't use netcdf, and then do not need it!

>
> I'm sending the patch since I don't know whether to push it to autoproj
> master or somewhere else.
>

You should have pushed both commits (the one for autoproj and the one
adding netcdf to the the OCL deps) at the same time, since now it breaks
compiling OCL master branch for autoproj users!

Charles.

>
> Ruben
> --
> Ruben Smits, Phd
> Chief Technology Officer
> Intermodalics BVBA
> +32479511786
> www.intermodalics.eu
>
> --
> Orocos-Dev mailing list
> Orocos-Dev [..] ...
> http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev
>
>

netcdf entry for autoproj

On Fri, Nov 30, 2012 at 8:39 AM, Charles Lesire-Cabaniols
<charles [dot] lesire [..] ...> wrote:
>
>
>
> 2012/11/22 Ruben Smits <ruben [dot] smits [..] ...>
>>
>> Hi,
>
>
> Hi Ruben,
>
>>
>>
>> Recently I started using the netcdf reporter, I noticed that it was once
>> disable due to it not compiling, this seems not to be the case anymore.
>> Therefor I enabled it again by default, added a system dependency on netcdf
>> in OCL's manifest. The attached patch for autoproj provides an entry for the
>> netcdf packages for ubuntu and debian.
>
>
> You should find a way to make this dependency optional: I think most of the
> people using OCL don't use netcdf, and then do not need it!
>
>>
>>
>> I'm sending the patch since I don't know whether to push it to autoproj
>> master or somewhere else.
>
>
> You should have pushed both commits (the one for autoproj and the one adding
> netcdf to the the OCL deps) at the same time, since now it breaks compiling
> OCL master branch for autoproj users!

Are you sure ? This patch is on gitorious...

Peter

netcdf entry for autoproj

2012/11/30 Peter Soetens <peter [..] ...>

> On Fri, Nov 30, 2012 at 8:39 AM, Charles Lesire-Cabaniols
> <charles [dot] lesire [..] ...> wrote:
> >
> >
> >
> > 2012/11/22 Ruben Smits <ruben [dot] smits [..] ...>
> >>
> >> Hi,
> >
> >
> > Hi Ruben,
> >
> >>
> >>
> >> Recently I started using the netcdf reporter, I noticed that it was once
> >> disable due to it not compiling, this seems not to be the case anymore.
> >> Therefor I enabled it again by default, added a system dependency on
> netcdf
> >> in OCL's manifest. The attached patch for autoproj provides an entry
> for the
> >> netcdf packages for ubuntu and debian.
> >
> >
> > You should find a way to make this dependency optional: I think most of
> the
> > people using OCL don't use netcdf, and then do not need it!
> >
> >>
> >>
> >> I'm sending the patch since I don't know whether to push it to autoproj
> >> master or somewhere else.
> >
> >
> > You should have pushed both commits (the one for autoproj and the one
> adding
> > netcdf to the the OCL deps) at the same time, since now it breaks
> compiling
> > OCL master branch for autoproj users!
>
> Are you sure ? This patch is on gitorious...
>

oups... effectively, it is! Works now ;)

>
> Peter
>