<div dir="ltr">Hi.<div><br></div><div>A few things to consider with respect to downstream maintenance support.</div><div><br></div><div>The configure script under the "for my $opt (@ARGV) {" loop covers a few typical items and any others cause the "die "Invalid option $opt\n";" line to be triggered. However, when in a Fedora/Enterprise Linux build environment, the "%configure" macro in the spec file adds several other switches (which work fine when building nginx directly).</div>
<div><br></div><div>Can the upstream please consider modifying  the configure script to allow for these?</div><div><br></div><div>A possiblity is:</div><div><br></div><div>for my $opt (@ARGV) {</div><div>...</div><div><br>
</div><div><div>    } elsif ($opt =~ /^--build=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--host=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>
    } elsif ($opt =~ /^--target=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--program-prefix=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--exec-prefix=(.*)/) {</div>
<div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--bindir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--sbindir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div>
<div><br></div><div>    } elsif ($opt =~ /^--sysconfdir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--datadir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div>
<br></div><div>    } elsif ($opt =~ /^--includedir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--libexecdir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br>
</div><div>    } elsif ($opt =~ /^--localstatedir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--sharedstatedir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div>
<br></div><div>    } elsif ($opt =~ /^--mandir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div><div>    } elsif ($opt =~ /^--infodir=(.*)/) {</div><div>        push @ngx_opts, $opt;</div><div><br></div>
<div><div>    } else {</div><div>        die "Invalid option $opt\n";</div><div>    }</div></div></div><div>}</div><div><br></div><div>May be even merge all the "dir" ones into "} elsif ($opt =~ /^--(.*)dir=(.*)/) {"</div>
<div><br></div><div>Also, can the upstream consider dropping version numbers from the folders in the "bundle" directory and record the version numbers in a README? Would save having to edit the spec file for every release.</div>
<div><br></div><div>Finally, can the upstream consider allowing the with luajit option to be selected but not to try to build luajit? I.E. set openresty up to use luajit but have the luajit build done separately and not try to replace this by the standard lua. I suppose the same applies to the standard lua. Basically, third party applications should ideally be built separately and we would look into creating seperate rpms for those. Can foresee some potential knotty issues to overcome with this but if distributing an rpm, then there are issues anyway.</div>
<div><br></div><div>Thanks</div><div><br></div><div><br></div></div>