[LITMUS^RT] [PATCH 0/2] Add module functionality to liblitmus

[Manohar Vanga]

Hmm it looks like my method has a limitation as well: you cannot name your tasks after existing rules in the makefile since I generate the rules from the object list.

Also, do you mean integrate the module rule with the compilation of all the tasks in bin/ ? I think the compilation of that works just fine and I don’t want to meddle with that. Anyway, let me think about a cleaner way to implement this without placing the restriction I mentioned above.

Thanks!

On Feb 5, 2014, at 11:00 PM, Manohar Vanga <mvanga at mpi-sws.org> wrote:

> 
>> We can still merge the modules patch. It doesn’t seem to add much complexity. However, it would be good if you could add a little README to explain how it’s supposed to be used. Further, why not move all all skeleton tasks into the example folder instead of duplicating just the non-threaded one?
> 
> Yes I’ll do this and resend the patches.
> 
>> For consistency, can you please also call the path to liblitmus simply LIBLITMUS (rather than adding a _DIR) suffix and provide a default like here:
>> 
>> 	https://github.com/LITMUS-RT/feather-trace-tools/blob/staging/Makefile#L8
> 
> Yes I already did this in my tree after noticing the convention.
> 
> Thanks!
> 
> —
> /mvanga
> 
> 
> _______________________________________________
> litmus-dev mailing list
> litmus-dev at lists.litmus-rt.org
> https://lists.litmus-rt.org/listinfo/litmus-dev


—
/mvanga