Opened 13 years ago

Closed 13 years ago

#15 closed defect (fixed)

Debian pacakges failing because it is building with --with-file-aio

Reported by: www.google.com/accounts/o8/id?id=AItOawncc3kSOjatWBTOYpfWpt5F-eKenzK848E Owned by: somebody
Priority: blocker Milestone:
Component: nginx-package Version: 1.0.x
Keywords: Cc:
uname -a: Linux 2.6.18.8-linode22 #1 SMP Tue Nov 10 16:12:12 UTC 2009 i686 GNU/Linux
nginx -V: nginx: nginx version: nginx/1.0.6
nginx: TLS SNI support enabled
nginx: configure arguments: --prefix=/etc/nginx/ --sbin-path=/usr/sbin/nginx --conf-path=/etc/nginx/nginx. conf --error-log-path=/var/log/nginx/error.log --http-log-path=/var/log/nginx/access.log --pid-path=/var/r un/nginx.pid --lock-path=/var/run/nginx.lock --http-client-body-temp-path=/var/cache/nginx/client_temp --h ttp-proxy-temp-path=/var/cache/nginx/proxy_temp --http-fastcgi-temp-path=/var/cache/nginx/fastcgi_temp --h ttp-uwsgi-temp-path=/var/cache/nginx/uwcgi_temp --http-scgi-temp-path=/var/cache/nginx/scgi_temp --user=ng inx --group=nginx --with-http_ssl_module --with-http_realip_module --with-http_addition_module --with-http _sub_module --with-http_dav_module --with-http_flv_module --with-http_gzip_static_module --with-http_rando m_index_module --with-http_secure_link_module --with-http_stub_status_module --with-mail --with-mail_ssl_m odule --with-file-aio --with-ipv6

Description

Hello,

Fresh install of Debian 6 and I added the following to sources.list:

deb http://nginx.org/packages/debian/ squeeze nginx
deb-src http://nginx.org/packages/debian/ squeeze nginx

When I am installing debian stable:

$ sudo apt-get install nginx
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
  nginx
0 upgraded, 1 newly installed, 0 to remove and 4 not upgraded.
Need to get 0 B/421 kB of archives.
After this operation, 893 kB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
  nginx
Install these packages without verification [y/N]? y
Selecting previously deselected package nginx.
(Reading database ... 71230 files and directories currently installed.)
Unpacking nginx (from .../nginx_1.0.6-1_i386.deb) ...
Setting up nginx (1.0.6-1) ...
Starting nginx : nginx.
$

I was visiting the website and it is taking time and the following text is in the error.log:

2011/09/14 23:25:56 [emerg] 10130#0: eventfd() failed (38: Function not impl
    emented)$
  2 2011/09/14 23:25:56 [alert] 10129#0: worker process 10130 exited with fatal
    code 2 and can not be respawn$

While searching online I saw this old post http://forum.nginx.org/read.php?2,152995 and the following changeset: http://trac.nginx.org/nginx/changeset/3071/nginx I recompiled nginx without --with-file-aio and it worked as planned.

Can you take out that compile option in Debian please?

Regarding the debian versioning, MTechnology from IRC pointed out:

23:36 < MTecknology> the -1 is a DEBIAN version, WE add that, they should be
                     publishing with -0, unless the want to override the -1
                     that we include
23:36 < MTecknology> in which case, they should use -2
23:36 < MTecknology> even then....
23:36 < MTecknology> they should use OUR version and APPEND
23:37 < MTecknology> 1.0.6-1  should be  1.0.6-0build1
23:37 < MTecknology> or just 1.0.6-0-1
23:38 < MTecknology> or.. 1.0.6-0~1  or  1.0.6-0ngx1  or  1.0.6-0xxyz1
23:40 < MTecknology> 1.0.6 is the nginx version, -0 -1 -2 -# is the Debian
                     version

Would be great to fix the stable build in the apt repository because at its current status, it doesn't even work .

Thanks. I placed the priority blocker because there is no way we can use this package.

Change History (3)

comment:1 by www.google.com/accounts/o8/id?id=AItOawncc3kSOjatWBTOYpfWpt5F-eKenzK848E, 13 years ago

More stuff regarding the packaging:

23:41 < MTecknology> If we didn't release -1, then others should match it with -0, if we released with -1, then they should
                     release with -1
23:41 < MTecknology> that keeps us all in sync
23:41 < MTecknology> now... to avoid same version conflicts, another string should be appended
23:42 < MTecknology> In the case of Ubuntu......
23:43 < MTecknology> if debian releases nothing, ubuntu would call it 1.0.6-0ubuntu1 for the first buisd of it, and
                     1.0.6-0ubuntu2 for an update
23:43 < MTecknology> if debian releases 1.0.6-1 then ubuntu would call their packages 1.0.6-1ubuntu1 or 1.0.6-1ubuntu2

comment:2 by Sergey Budnevitch, 13 years ago

Status: newaccepted

Linux 2.6.18.8-linode22 is not a stock debian kernel, and it is evident from error.log that it has no aio support unlike debian one. aio on linux despite some drawbacks do improve performance on certain load scenarios, so --with-file-aio will be kept in default package as it works with stock kernel. But i'll build separate package without aio for next release.

As for package name scheme MTecknology describe naming for derived from debian packages, but nginx package was build from scratch and independent from debian one. If we actually violate some debian policy, please show me link to it and I'll change naming scheme

comment:3 by Sergey Budnevitch, 13 years ago

Resolution: fixed
Status: acceptedclosed

New 1.0.7 package should work on kernel without aio support (r4165).

Note: See TracTickets for help on using tickets.