search

Tinkoff / Nginx-builder

A tool to build deb or rpm package of required Nginx version from the source code, with the ability to connect third-party modules. Nginx parameters are set in the yaml configuration file.
Apache License 2.0
153 stars 30 forks source link

Cannot find nginx source rpm(SRPM) with version 1.16.0 in url http://nginx.org/packages/mainline/centos/7/SRPMS #11

Closed patsevanton closed 5 years ago

patsevanton commented 5 years ago

Пытаюсь собрать 16 версию nginx

cat config.yaml 
nginx_version: 1.16.0
output_package: rpm
modules:
  - module:
      name: nginx-dav-ext-module
      git_url: https://github.com/arut/nginx-dav-ext-module.git
      git_tag: v3.0.0
./main.py build -f config.yaml 
builder - INFO - Parse yaml file: config.yaml
builder - INFO - Download scripts for build rpm package
builder - INFO - Running command: rpmdev-setuptree
builder - INFO - 
builder - INFO - Downloading nginx src...
builder - ERROR - Cannot find nginx source rpm(SRPM) with version 1.16.0 in url http://nginx.org/packages/mainline/centos/7/SRPMS

Может добавить использование https://nginx.org/download/nginx-1.16.0.tar.gz ? Например, код взять из https://nginx.org/download/nginx-1.16.0.tar.gz, а остальные файлы с с других SRPM ( http://nginx.org/packages/mainline/centos/7/SRPMS/nginx-1.15.9-1.el7_4.ngx.src.rpm )?

patsevanton commented 5 years ago

Если использовать конфиг из статьи https://habr.com/ru/company/tinkoff/blog/452336/ с rpm, то будет похожая ошибка

./main.py build -f config.yaml 
builder - INFO - Parse yaml file: config.yaml
builder - INFO - /usr/src/nginx dir already exist
builder - INFO - Download scripts for build rpm package
builder - INFO - Running command: rpmdev-setuptree
builder - INFO - 
builder - INFO - Downloading nginx src...
builder - ERROR - Cannot find nginx source rpm(SRPM) with version 1.14.1 in url http://nginx.org/packages/mainline/centos/7/SRPMS
patsevanton commented 5 years ago

Находятся они в другом месте:

http://nginx.org/packages/centos/7/SRPMS/

patsevanton commented 5 years ago

Попробую сделать PR

patsevanton commented 5 years ago

PR https://github.com/TinkoffCreditSystems/Nginx-builder/pull/13

javdet commented 5 years ago

Добрый день, спасибо за PR. Эта же проблема присутствует для deb пакетов. Готовлю исправления для обоих случаев.

patsevanton commented 5 years ago

Я могу попробовать сделать PR. Что нужно поправить в моем PR?

javdet commented 5 years ago

Сделал единый подход к выбору stable и mainline версий для deb и rpm. Изменения уже в мастере #15