Openwrt makefile#61
Conversation
Thanks for aabc#57 to bc-davidb@github.
dkms runs ./configure --from-dkms-conf=$kernel_source_dir. Without this ./configure fails with invalid option. Change the check to match the one at the top of the file.
place in package/network/ipt-netflow/
select using menuconfig
rebuild with
make package/network/ipt-netflow/{clean,prepare,configure,compile,install}
aabc
left a comment
aabc
left a comment
There was a problem hiding this comment.
Can you rework this patch to be applied directly to Makefile.in?
aabc
left a comment
aabc
left a comment
There was a problem hiding this comment.
Even though both project homepages are valid (github and SF) github one is now preferred.
|
Патч вынесен для того чтоб компилировать версию 2.2. Если я применяю его сразу к исходникам, то при скачивании с сорсфоржа или тага с гитхаба - для сборки будет использоваться непатченная версия. Может быть в следующей версии этот патч не понадобится. Можете применить отсоединенный патч к исходникам (если он не ломает сборку на других системах), а после его применения поставить в таг новую версию ? Как разберусь с опциями configure(сейчас hardcoded) и переделаю на гитхаб. |
|
То есть этот пекедж не будет собирать версию из git, а предыдущий релиз? |
Applied alex-eri's patch openwrt/patches/310-Makefile_crosscompile.patch to compile for OpenWrt. Pull #61.
|
Закоммитил это изменение Makefile в db665c0. |
|
Для релиза 2.3 надо подождать, чтоб убедиться, что последние изменения ничего не ломают, прогнать тесты как минимум. |
|
My tests passed all good with latest git head. Probably, it may be time to release v2.3. |
Change sources location Possible options for ./configure #TODO: --enable-sampler=hash --promisc-mpls=N #TODO: snmp install, dkms install
|
Added configuration menu and changed source location to github. Помню как-то тут можно откатить коммиты и добавить в них файлы.. если хорошо разбираешься в гите, то можешь попробовать. я не осилю... а так прицепить бы этот пулреквест к версии 2,2 Когда будете делать релиз 2.3 или 2.2.1 удалите патч и поменяйте версию в мэйкфайле |
|
Я замержил ваши изменения https://github.com/aabc/ipt-netflow/commits/master |
|
Почему в openwrt/Makefile При релизе 2.3 что делать - достаточно поменять |
|
URL ссылка на сайт проекта, ни на что не влияет. для документации там, в меню можно посмотреть инфо о пакете. PKG_SOURCE_URL - оттуда сборщик качает исходник и перепаковывает в архив. да, только надо в одном коммите всё зарелизить мне кажется. чтоб на ветке 2.2 остался с патчем, а на 2.3 уже без. |
Что это значит? Перед релизом 2.3 я файл патча удалю как не нужный и всё или не всё? Сейчас патч есть, а на 2.3 патч накладывать смысла нет. В инструкциях в вашем Readme.md я ничего про накладывание патчей не вижу. Сказано
Никакие патчи не упоминаются, значит они и не нужны. В Makefile команд накладывающих патч тоже не вижу. |
|
да, все правильно. так и говорю удалить перед коммитом.. там сотня мейкфайолв которые собираются в один в 1500 строк перед сборкой. патчи накладываются при условии если они там лежат рядом. я думаю не стоит заморачиваться с ними вообще если релиз скоро.. |
|
I plan to release 2.3 real soon, after merging this readme update (and deleting patch). |
|
Я сделал Затем Если вы не против всех этих изменений и если я ничего не забыл - то так все и пойдёт в релиз 2.3. |
|
ok |
aabc
force-pushed
the
master
branch
13 times, most recently
from
daeca11 to
1d72324
Compare
3 participants
Place files "package/network/ipt-netflow/" in OpenWRT build and select in menuconfig.
Builds on Chaos Calmer, fixes #39 .