From mboxrd@z Thu Jan 1 00:00:00 1970 From: Masahiro Yamada Date: Sat, 27 Aug 2016 01:47:14 +0900 Subject: [U-Boot] [PATCH 02/42] moveconfig: Add an option to commit changes In-Reply-To: References: <1472057546-10360-1-git-send-email-sjg@chromium.org> <1472057546-10360-3-git-send-email-sjg@chromium.org> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 2016-08-25 21:04 GMT+09:00 Simon Glass : > Hi Masshiro, > > On 24 August 2016 at 19:40, Masahiro Yamada > wrote: >> 2016-08-25 1:51 GMT+09:00 Simon Glass : >>> The moveconfig tool is quite clever and generally produces results that >>> are suitable for sending as a patch without further work. The main required >>> step is to add the changes to a commit. >>> >>> Add an option to do this automatically. This allows moveconfig to be used >>> from a script to convert multiple CONFIG options, once per commit. >>> >>> Signed-off-by: Simon Glass >> >> >> >> Isn't it possible to do this in your script? >> >> >> BTW, I assume you want this feature >> for convert & git-commit per option. >> >> >> In my opinion, you do not have to split to >> per-option patches. >> >> >> You can move multiple options at once. >> >> >> tools/moveconfig SPL_DM SPL_RSA SPL_CRYPTO_SUPPORT ... > > Yes of course, but as you can see from your other comments, some > changes are simple and others result in comments and rework. It seems > better to deal with each change individually unless they really are > simple. > > Most of these changes affect a lot of boards. If there is a run-time > breakage it helps a lot if a bisect can pinpoint the change exactly. > >> >> >> Then, git commit and list all the moved options in the commit log. > > OK. I think the moveconfig tool should handle committing changes since > it can do a better job of creating a sensible commit message. No, I don't like this patch. Anyway, a good practice for moving options is to create a Kconfig entry and move configs in the same commit, unlike this series. So, you need some manual interventions to build up patches. -- Best Regards Masahiro Yamada