From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM autolearn=ham autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=subject:to:references:from:message-id:date:user-agent:mime-version :in-reply-to:content-transfer-encoding:content-language; bh=XZ08+9K58/xHJxt+FWjnlBX/Yx/jzBbnR/symvfHWek=; b=ll9PYK7R8jBRwkNHhY3uZ+GMsT/Lgn72h618d89aXV6g53DoyLnfrd628ZgohkUWJ+ x0wzIX70BcLlh23OMOpXmURZJdwmS+teZ6n+oT+ZuUiV1CAdjxMXP9mRCxAX0c37K2SF rC/EYxQsqYiQ2CY5bwvirxYkqbE2VEWN3Sip1wwbMDe9H3ofgGLOgC/h+QTyBfRzp8Hz 0UJe3RDO8tQhEMVTO2+tgvaqJQG5BDkvdD72vPlnQlQIRDZzVO3CZ3EjvU4A2S0CRk6d bDFV8+jJO3gved0QgXYAskPqbIgmguCpnTUN/Fd7jB5OgL0tU0qGyeB1cE2U/GEw811t 5yVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=XZ08+9K58/xHJxt+FWjnlBX/Yx/jzBbnR/symvfHWek=; b=EITTOG5gm/umHjT6KDI6QRqSogIKiS90JeYHjf5zELhdFbdCdZYIxiD0DuoZoGYA9J MQH1rD/bAu7DrIdBP9jYCss8cC81JmCqYpQtA9f0eQglRwxST/eK4O+DyYkwNBYEbTjV GEXuiNDkVwzXomMLco3il4hRNrnH71kieK7W5JBgJsSH4Aj9UzxvWeXnSD9bEuAwiON8 zRKA9Qxw0w33zIep75N4+xQEAWqZFn1YdfKvH17eRNB/dkaTa667yRu6V9mRCLqcR3Ps 9gSAoj5oyu9fEXQX2LuyPOWXmTHEEr//7GRJy88AwIKlEn3S3Zz/F4qgoSvOxZ9hvWD5 Rtiw== X-Gm-Message-State: AOAM532s8o6yuqpj7QDWLAqvnf9EXmpr/HCXw8iKdoLwiJgzpk5jY9GX 0P88FOm8vkshBUMfO3CNCvZELPGTazU= X-Google-Smtp-Source: ABdhPJytkMUjFR9wUoXj2H1OxNOomL/41D2jmLao63XVumc1LsnSLk4umqs5nX6OXaqxoQ0LVCFCRw== X-Received: by 2002:ac2:4e0d:: with SMTP id e13mr20640952lfr.560.1632686979991; Sun, 26 Sep 2021 13:09:39 -0700 (PDT) To: make-initrd@lists.altlinux.org References: <20210924190822.hkqkjmcsj3u3zazv@example.org> <20210926142921.qnmrkydotjtpq664@example.org> From: Leonid Krivoshein Message-ID: <4aa5ac9b-ae30-2e1d-171a-fb92591c5afb@gmail.com> Date: Sun, 26 Sep 2021 23:09:39 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: <20210926142921.qnmrkydotjtpq664@example.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: ru Subject: Re: [make-initrd] [PATCH v1 00/41] fork pipeline X-BeenThere: make-initrd@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: make-initrd@lists.altlinux.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 26 Sep 2021 20:09:43 -0000 Archived-At: List-Archive: 26.09.2021 17:29, Alexey Gladkov пишет: > On Sat, Sep 25, 2021 at 12:59:27AM +0300, Leonid Krivoshein wrote: >> 24.09.2021 22:08, Alexey Gladkov пишет: >>>> На первой итерации, чтобы не флудить, сделал поменьше коммитов. Всё равно >>>> вспомнить "что за чем менялось" уже невозможно. На следующих итерациях >>>> постараюсь сделать это более понятным. Поэтому первые три коммита позволяют >>>> понять порядок перемещения, переименования и что создано нового. Заранее >>>> извиняюсь за порядок и оформление коммитов, готов исправиться. >>> Я начал читать, но так как кода очень много не успею за один день всё >>> осилить. Я буду ещё комментировать. >> Да это понятно, смотри, по возможности. А я тут подумал, что bootchain не >> просто же так форкался и переделывался, это не только демон и API -- >> основная часть его "клиентов" дойдёт до апстрима следом, но сейчас этого нет >> в удобном тебе виде. Может помочь черновик документации, что я выслал. Также >> прилагаю лог самой проблемной загрузки по FTP, сделанный в результате >> сегодняшней проверки отправленного в апстрим. Могу выслать такой же с >> локальной загрузкой, если надо. Где-то это может понять логику работы кода и >> подход к отладочным сообщениям. Ну и на вопросы буду рад ответить! > Я дальше смотреть пока не буду. Пожалуйста переделай патчи. Конечно, переделаю. > Твои патчи не > разбиты логически, в них нет описания зачем делаются те или иные > изменения, в README я не увидел описаний. На первом этапе я хотел показать изменения pipeline, в целом, но сделать более читабельной историю и не флудить большим числом изменений. Мне и самому надо было понять, так как делались эти изменения очень давно и история нигде не сохранилась. Необходимость писать правильные commit message я конечно понимаю. Сейчас их написание притормозит процесс. И ещё более сильно притормозит, если придётся писать README к каждой фиче на английском языке -- черновик документации на русском более 40 страниц. Кстати, при разделении pipeline на 4 фичи должно получиться 4 README, включая исходный вариант, который я не трогал. Я вот не понял суть замечаний к тем двум README, что попали в bootchain-getimage и bootchain-waitdev. А про README из bootchain-core уже написал ранее. > Форк + rename можно сделать за один коммит. После этого уже идут > содержательные коммиты, которые приносят твой новый функционал или меняют > поведение. > > Сейчас же есть пачка "fork pipeline: * modified", которые либо делают > переименование, либо за одно что-то правят. -- Best regards, Leonid Krivoshein.