+++ Matěj Kubička [2016-03-19 04:11 +0100]:
> Attached. @dev-SVN-r20216. ~Matej.

Thanks for that fix (I do ARM porting for a living so it's always good
to see people fixing things upstream).

But then I wondered how come this is already built for 3 ARM arches on debian?
https://buildd.debian.org/status/package.php?p=sumo

And indeed an equivalent patch (for the flags issue) is already in the
debian packaging:
https://sources.debian.net/src/sumo/0.25.0%2Bdfsg1-2/debian/patches/40_drop_some_buildflags.patch/
(Sensible defaults exist on each arch so just dropping this 'special
x86 flags' code is best there. Interesting question what the best
upstream behaviour is - will people actually be building (on x86) with
toolchains that don't turn SSE on by default anyway? If so then
enabling neon on arm and arm64 and altivec on mips and so on would be
better than just special-casing x86).

This sugests that no-one has looked at the patches distros are
carrying to see if they should be upstream.

All 5 debian patches are here:
https://sources.debian.net/src/sumo/0.25.0%2Bdfsg1-2/debian/patches/

And everything except the openlayers patch looks suitable for upstream to me.

So you might want to review those and include them if they seem good, then 
Debian can drop them.

If you have any further questions about arm support or being a good
upstream from a distro and 'the world is more than just x86' point of
view, I am happy to help.

Wookey
-- 
Principal hats:  Linaro, Debian, Wookware, ARM
http://wookware.org/

Attachment: signature.asc
Description: Digital signature

------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785231&iu=/4140
_______________________________________________
sumo-user mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sumo-user

Reply via email to