[PATCH] e2k arch support

[Michael Shigorin]

Hello,
lua 5.3 builds for me on the Russian Elbrus (e2k) architecture
with the (quite trivial) patch attached.

[Michael Shigorin]

Post by Michael Shigorin
lua 5.3 builds for me on the Russian Elbrus (e2k) architecture
with the (quite trivial) patch attached.

Guys, is it enough to send in the patch and wait for a while
or should I do something particular to make sure these two lines
land into the lua source tree?

TIA :)

[Andrew Gierth]

lua 5.3 builds for me on the Russian Elbrus (e2k) architecture with
the (quite trivial) patch attached.

Michael> Guys, is it enough to send in the patch and wait for a while
Michael> or should I do something particular to make sure these two
Michael> lines land into the lua source tree?

I think you're confused about what exactly you are patching - your diff
does not match anything in the lua 5.3.5 source distribution.

[Michael Shigorin]

Post by Andrew Gierth

lua 5.3 builds for me on the Russian Elbrus (e2k) architecture with
the (quite trivial) patch attached.

Michael> Guys, is it enough to send in the patch and wait for a while
Michael> or should I do something particular to make sure these two
Michael> lines land into the lua source tree?
I think you're confused about what exactly you are patching -

That was 5.3.4, apparently.

Post by Andrew Gierth
your diff does not match anything in the lua 5.3.5 source distribution.

...and this was what I've missed, thank you; 5.3.5 just builds.

I-will-check-for-the-latest-version :)

[Andrew Gierth]

Michael> Guys, is it enough to send in the patch and wait for a while
Michael> or should I do something particular to make sure these two
Michael> lines land into the lua source tree?

Post by Andrew Gierth
I think you're confused about what exactly you are patching -

Michael> That was 5.3.4, apparently.

It doesn't match anything in the 5.3.4 sources either?

[Michael Shigorin]

Post by Andrew Gierth

Post by Andrew Gierth
I think you're confused about what exactly you are patching -

Michael> That was 5.3.4, apparently.
It doesn't match anything in the 5.3.4 sources either?

Argh, finally realized that it was a packaging addition
in the first place; originating from this one I think:
https://src.fedoraproject.org/rpms/lua/blob/master/f/luaconf.h

Sorry for the noise, you just do fine already! :)