From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 12 Jan 2022 13:53:53 +0100 From: Alexey Gladkov To: ALT Linux Team development discussions Message-ID: <20220112125353.eggdcwcwrviq5hpm@example.org> References: <20220111111209.0ec47280@tower> <3005468.b3WyR9EExj@zerg.malta.altlinux.ru> <20220112002655.d7kqtjsj32hv5j23@altlinux.org> <20220112003920.GG32049@altlinux.org> <20220112122051.GB7960@altlinux.org> <20220112124003.xlsrfqt4iblg7v5u@example.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220112124003.xlsrfqt4iblg7v5u@example.org> Subject: Re: [devel] =?utf-8?q?PoC=3A_gear-submodule-update_Ex=3A_=D0=9D=D0=BE?= =?utf-8?b?0LLQsNGPINGB0YXQtdC80LAg0LLQtdC00LXQvdC40Y8g0LjRgdGF0L7QtNC9?= =?utf-8?b?0LjQutC+0LIg0Y/QtNGA0LA=?= X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 12 Jan 2022 12:53:55 -0000 Archived-At: List-Archive: List-Post: On Wed, Jan 12, 2022 at 01:40:03PM +0100, Alexey Gladkov wrote: > On Wed, Jan 12, 2022 at 03:20:51PM +0300, Dmitry V. Levin wrote: > > On Wed, Jan 12, 2022 at 03:39:20AM +0300, Dmitry V. Levin wrote: > > > On Wed, Jan 12, 2022 at 03:26:55AM +0300, Vitaly Chikunov wrote: > > > > On Tue, Jan 11, 2022 at 12:10:52PM +0300, Sergey V Turchin wrote: > > > > > On Tuesday, 11 January 2022 11:55:20 MSK Paul Wolneykien wrote: > > > > > > > > > > [...] > > > > > > Возможно, это повод для FR к gear: чтобы он поддерживал в rules ссылки на > > > > > > внешние репозитории или подмодули. > > > > > Возможно, с появлением gear все этого уже ждут. ;-) > > > > > https://bugs.altlinux.org/17914 > > > > > > > > Возможны разные попытки решения этой проблемы. > > > > Вот Proof of Concept code > > > > gitery:/people/vt/private/gear-submodule-update.git > > > > который делает фиктивный merge всем submodules и меняет rules и spec > > > > так чтоб они разтаривались при сборке. > > > > > > > > Это решение чисто экспериментальное (as in no warranty), но, возможно, > > > > оно вдохновит кого-то на более правильное решение. > > > > > > Есть ещё > > > https://git.altlinux.org/people/ldv/packages/?p=gear.git;a=shortlog;h=refs/heads/legion/patchset/submodule-support/v5 > > > но я его не смог дочитать. > > > > Вроде бы дочитал, отредактировал, запушил в > > https://git.altlinux.org/people/ldv/packages/?p=gear.git;a=shortlog;h=submodules > > + if [ -s "$workdir/listed_bundles" ]; then > + sort -u -o "$workdir/listed_bundles"{,} > + >> "$workdir/appended_bundles" > > Кажется что-то пошло не так при редактировании. > Разве не должно быть > > sort -u -o "$workdir/listed_bundles"{,} \ > >> "$workdir/appended_bundles" > > ? > > Отредактированные коммиты точно тесты проходят ? ))) > > + sort -u -o "$workdir/appended_bundles"{,} > + comm -23 "$workdir/listed_bundles" \ > + "$workdir/appended_bundles" \ > + > "$workdir/unused_submodules" > + if [ -s "$workdir/unused_submodules" ]; then > + lineno=0 > + local rules='.gear/submodules/list' > + local bundle sm_hash sm_path dummy > + while read -r bundle sm_hash sm_path dummy; do { > + lineno=$(($lineno + 1)) > + grep -Fxqse "$lineno" "$workdir/unused_submodules" || > + continue > + rules_info "unused bundle: $sm_path" > + } < /dev/null; done < "$workdir/submodules_list" > > Может лучше > > grep -Fxqse "$lineno" "$workdir/unused_submodules" < /dev/null > > а не этот ужасный блок ? В других местах я вижу ты тоже такую конструкцию добавил. Я бы сделал: exec 4