Geontech / meta-redhawk-sdr

REDHAWK SDR Layer for Yocto/OpenEmbedded -based deployments
http://geontech.com/getting-started-with-meta-redhawk-sdr/
GNU Lesser General Public License v3.0
9 stars 6 forks source link

Disable CORBA LongDouble by default #55

Closed abogani closed 4 years ago

abogani commented 4 years ago

Signed-off-by: Alessio Igor Bogani alessio.bogani@elettra.eu

btgoodwin commented 4 years ago

@abogani,

Thank you for the contribution. We've started following the yocto release branch setup, so master is actually pretty far behind now (we're keeping it for some historical projects, but that's it). This patch has been pulled into the thud and rocko branches already with a tweak to be for all 32-bit ARM only vs. specifically zynq (Xilinx) parts as before:

https://github.com/Geontech/meta-redhawk-sdr/blob/rocko/recipes-deps/omniorb/omniorb_4.2.3.bb#L47

Does that fully fix it for you?

abogani commented 4 years ago

@btgoodwin,

[...]

Thank you for the contribution. We've started following the yocto release branch setup, so master is actually pretty far behind now (we're keeping it for some historical projects, but that's it).

It isn't completely true: in the Yocto project the master branch is the most updated version of the code (actually it contains what become zeus release soon). There is also master-next but it is more a branch for Yocto developer than for everyone else (and for this reason it is rebased very often).

This patch has been pulled into the thud and rocko branches already with a tweak to be for all 32-bit ARM only vs. specifically zynq (Xilinx) parts as before:

https://github.com/Geontech/meta-redhawk-sdr/blob/rocko/recipes-deps/omniorb/omniorb_4.2.3.bb#L47

Does that fully fix it for you?

Yes it does but I don't like it actually: I have other architectures in the field so I would like to avoid incompatibility because some naive programmer choose to use it. Moreover we are using warrior and we have already planned to switch to zeus when it'll became officially available.

Thanks for all!

btgoodwin commented 4 years ago

Nice to hear REDHAWK builds on warrior! Were there any major changes or challenges?

I would be happy to accept EXTRA_OECONF appends for those other architectures if you can share them.

On Fri, Sep 27, 2019 at 9:29 AM Alessio Igor Bogani < notifications@github.com> wrote:

@btgoodwin https://github.com/btgoodwin,

[...]

Thank you for the contribution. We've started following the yocto release branch setup, so master is actually pretty far behind now (we're keeping it for some historical projects, but that's it).

It isn't completely true: in the Yocto project the master branch is the most updated version of the code (actually it contains what become zeus release soon). There is also master-next but it is more a branch for Yocto developer than for everyone else (and for this reason it is rebased very often).

This patch has been pulled into the thud and rocko branches already with a tweak to be for all 32-bit ARM only vs. specifically zynq (Xilinx) parts as before:

https://github.com/Geontech/meta-redhawk-sdr/blob/rocko/recipes-deps/omniorb/omniorb_4.2.3.bb#L47

Does that fully fix it for you?

Yes it does but I don't like it actually: I have other architectures in the field so I would like to avoid incompatibility because some naive programmer choose to use it. Moreover we are using warrior and we have already planned to switch to zeus when it'll became officially available.

Thanks for all!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Geontech/meta-redhawk-sdr/pull/55?email_source=notifications&email_token=ABJLAS4KKSKCA3LX2K2A3STQLYDFHA5CNFSM4I3DDSKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7Y4THA#issuecomment-535939484, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJLAS3MGFVHF43SOTTGIDDQLYDFHANCNFSM4I3DDSKA .

abogani commented 4 years ago

@btgoodwin,

Actually it also works on current master branch of Yocto (that will become Zeus soon). I think that has proven your good work on OmniORB recipe!

btgoodwin commented 4 years ago

Nice! Thanks!

On Sun, Sep 29, 2019 at 4:25 PM Alessio Igor Bogani < notifications@github.com> wrote:

@btgoodwin https://github.com/btgoodwin,

Actually it also works on current master branch of Yocto (that will become Zeus soon). I think that has proven your good work on OmniORB recipe!

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/Geontech/meta-redhawk-sdr/pull/55?email_source=notifications&email_token=ABJLASYLEEKQTN2ET3YWUVTQMEFMVA5CNFSM4I3DDSKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7355AY#issuecomment-536338051, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJLAS64VJB26GOPWTR57PLQMEFMVANCNFSM4I3DDSKA .