Re: [Paparazzi-devel] Dependency problem

[Ben Laurie]

On 29 October 2013 12:03, Christophe De Wagter <address@hidden> wrote:
> paparazzi (or actually make) tracks dependencies quite well, however, make
> does not understand the xml-configuration well.

Hmm. When I edit gps_ubx.h and do:

% make AIRCRAFT=ardrone2_raw ap.compile

it doesn't recompile anything. It should.

>
>  - So if for instance you change your xml and add a <define ... > somewhere,
> make does not understand which control or driver files are affected. You
> then MUST clean before recompile.
>  - Even worse: if you change the board file to another arch in the middle of
> some work, it will only rebuild the changed c files so you can have object
> files compiled with different architectures being linked (which does
> fortunately not work)
>  - And if you change a module.xml, the build system does not necessarly know
> it should recompile (whjile for subsystems it does know as it is makefile)
>
> Our rules of thumb:
>  - make clean anytime you need to be really sure
>  - certainly clean after every xml change
>
> PS: thanks for the pull request.
>
> -Christophe
>
>
> On Tue, Oct 29, 2013 at 12:01 PM, Ben Laurie <address@hidden> wrote:
>>
>> On 28 October 2013 12:50, Felix Ruess <address@hidden> wrote:
>> > Hi Ben,
>> >
>> > seems that this is because the gps_ubx_ucenter.c file is INCLUDED in
>> > subsystems/gps/gps_ubx.c Line 271 rather than being added as a normal
>> > source
>> > file.
>> > While this was probably easier to implement (to allow access to ubx
>> > internals to the ucenter module) it's not very nice.
>> > Christophe may have more details as he wrote the gps_ubx_ucenter module.
>>
>> I am sending a pull request that fixes this.
>>
>> In general, though, it seems paparazzi doesn't track dependencies. Am
>> I right, or am I missing something?
>>
>> >
>> > Cheers, Felix
>> >
>> >
>> > On Sat, Oct 26, 2013 at 7:55 PM, Ben Laurie <address@hidden> wrote:
>> >>
>> >> If I modify gps_ubx_center.c, then rebuild ardrone2_raw, it does not
>> >> recompile that file...
>> >>
>> >> _______________________________________________
>> >> Paparazzi-devel mailing list
>> >> address@hidden
>> >> https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
>> >
>> >
>> >
>> > _______________________________________________
>> > Paparazzi-devel mailing list
>> > address@hidden
>> > https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
>> >
>>
>> _______________________________________________
>> Paparazzi-devel mailing list
>> address@hidden
>> https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
>
>
>
> _______________________________________________
> Paparazzi-devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
>