Re: Patch for typgen, adding -x option
Peter Soetens <[email protected]>
| Newsgroups | gmane.science.robotics.orocos.devel |
|---|---|
| Message-ID | <CAMYDobUyRh5UzPKG5p2TdGjosvyeUiPnZNizswzry0_2bf4Y7w@mail.gmail.com> |
On Fri, Feb 1, 2013 at 2:01 PM, Sylvain Joyeux <[email protected]> wrote: > On 02/01/2013 12:19 PM, Peter Soetens wrote: >> >> I wanted to pass typegen an option that informs it for which >> OROCOS_TARGET it needs to find the .pc files, while still keeping the >> 'normal' name for looking up the .tlb files. >> >> The minor patch in attachment adds such an option, -x <target> >> >> Ok to apply ? > > You should create a toplevel imports array, gather all the -i options and > resolve them at once after the call to parser.parse. Otherwise, the place of > the -x option will matter. I know, the -h docs of -x mention this...I'll see if I have the ruby skills to implement this > > Moreover, there is already an Orocos::Generation.orocos_target attribute > that picks up the OROCOS_TARGET envvar automatically. You should probably > use that by default, and set it within the -x option. I see, but you mean: override it with the -x option ? > > Sylvain Peter -- Orocos-Dev mailing list [email protected] http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev