From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 12 Jan 2022 13:40:03 +0100 From: Alexey Gladkov To: ALT Linux Team development discussions Message-ID: <20220112124003.xlsrfqt4iblg7v5u@example.org> References: <20220111111209.0ec47280@tower> <3005468.b3WyR9EExj@zerg.malta.altlinux.ru> <20220112002655.d7kqtjsj32hv5j23@altlinux.org> <20220112003920.GG32049@altlinux.org> <20220112122051.GB7960@altlinux.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220112122051.GB7960@altlinux.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:40:05 -0000 Archived-At: List-Archive: List-Post: 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 а не этот ужасный блок ? + fi + fi > но всё равно у меня нет полной уверенности, что предпоследний коммит, > который обрабатывает .gear/submodules/list в gear, делает это совершенно > безопасно. > > > -- > ldv > _______________________________________________ > Devel mailing list > Devel@lists.altlinux.org > https://lists.altlinux.org/mailman/listinfo/devel -- Rgrds, legion