Welcome! Log In Create A New Profile

Advanced

Re: [PATCH 0 of 8] Allow crossbuilding and Android support

Martin Ertsaas
July 29, 2014 09:42AM
On 07/29/14 15:34, Maxim Dounin wrote:
> Hello!
>
> On Tue, Jul 29, 2014 at 03:02:21PM +0200, Martin Ertsaas wrote:
>
>> On 07/29/14 14:57, Maxim Dounin wrote:
>>> Hello!
>>>
>>> On Tue, Jul 29, 2014 at 08:24:21AM +0200, Martin Ertsaas wrote:
>>>
>>>> I haven't the nginx know how to evaluate this, but have anyone had the
>>>> time to look at these patches?
>>> I've looked into these patches, and I don't think they worth
>>> considering. Looks like a set of dirty hacks.
>>>
>> Do you have any specifics to what you don't like so we could look into
>> changing them? Or are you just not interested in crossbuild support for
>> nginx?
> Crossbuilding was discussed more than once here. In short: while it's
> something nice to have, it's not something required from our point
> of view; therefore patches to add it should be clean, simple, and
> extandable; no hacks will be considered.
>

I get that, and we do want to do that. We therefor also would appreciate
some critisism, and also pointers to how we else could solve things in
your build system. For us cross building is important, and we are
willing to spend time both adding it for ourselves and getting it
upstreamed. We did not expect that these patches would be accepted as
they were, but were hoping for some response as to which parts needs to
be fixed from you guys that knows the nginx build system a lot better
than we do.

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH 0 of 8] Allow crossbuilding and Android support

Per Olav Høydahl Ohme 647 July 18, 2014 08:18AM

[PATCH 2 of 8] Added new root prefixes option

Per Olav Høydahl Ohme 297 July 18, 2014 08:18AM

[PATCH 6 of 8] Added new run-time prefix option for configure files

Per Olav Høydahl Ohme 342 July 18, 2014 08:18AM

[PATCH 3 of 8] Include features which can be compiled when crossbuilding

Per Olav Høydahl Ohme 303 July 18, 2014 08:18AM

Re: [PATCH 0 of 8] Allow crossbuilding and Android support

Martin Ertsaas 327 July 29, 2014 02:24AM

Re: [PATCH 0 of 8] Allow crossbuilding and Android support

Maxim Dounin 296 July 29, 2014 09:00AM

Re: [PATCH 0 of 8] Allow crossbuilding and Android support

Martin Ertsaas 291 July 29, 2014 09:02AM

Re: [PATCH 0 of 8] Allow crossbuilding and Android support

Maxim Dounin 295 July 29, 2014 09:36AM

Re: [PATCH 0 of 8] Allow crossbuilding and Android support

Martin Ertsaas 344 July 29, 2014 09:42AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 138
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready