diff mbox series

[v2] scripts/git.orderfile: Display meson files along with buildsys ones

Message ID 20200907161222.41915-1-philmd@redhat.com
State New
Headers show
Series [v2] scripts/git.orderfile: Display meson files along with buildsys ones | expand

Commit Message

Philippe Mathieu-Daudé Sept. 7, 2020, 4:12 p.m. UTC
Since commit a56650518f5 ("configure: integrate Meson in the build
system") we replaced many Makefile by Meson files. Adapt the
git.orderfile script to display the new file at the same position.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 scripts/git.orderfile | 1 +
 1 file changed, 1 insertion(+)

Comments

Stefano Garzarella Sept. 8, 2020, 9:17 a.m. UTC | #1
On Mon, Sep 07, 2020 at 06:12:22PM +0200, Philippe Mathieu-Daudé wrote:
> Since commit a56650518f5 ("configure: integrate Meson in the build
> system") we replaced many Makefile by Meson files. Adapt the
> git.orderfile script to display the new file at the same position.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  scripts/git.orderfile | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/scripts/git.orderfile b/scripts/git.orderfile
> index 73fd818d7f3..a021afc2d51 100644
> --- a/scripts/git.orderfile
> +++ b/scripts/git.orderfile
> @@ -18,6 +18,7 @@ docs/*
>  configure
>  Makefile*
>  *.mak
> +meson.build
>  
>  # qapi schema
>  qapi/*.json
> -- 
> 2.26.2
> 
> 

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
Paolo Bonzini Sept. 8, 2020, 11:48 a.m. UTC | #2
On 08/09/20 12:02, Laszlo Ersek wrote:
> Possibly related (not sure if there's another patch pending for it
> already): if you run a "make clean" in the source tree, and have the
> Meson submodule initialized, the "make clean" will delete files from the
> Meson submodule.
> 
> find . \( -name '*.so' -o -name '*.dll' -o -name '*.[oda]' \) -type f \
> 
> "*.d" matches various meson files.
> 
> (Again, apologies if this is already known; I understand this is
> probably the worst context to report his in...)

Is this running "make clean" from an old commit (i.e. as in an in-tree
build) after having checked out the meson submodule?

Paolo
Laszlo Ersek Sept. 8, 2020, 12:30 p.m. UTC | #3
On 09/08/20 13:48, Paolo Bonzini wrote:
> On 08/09/20 12:02, Laszlo Ersek wrote:
>> Possibly related (not sure if there's another patch pending for it
>> already): if you run a "make clean" in the source tree, and have the
>> Meson submodule initialized, the "make clean" will delete files from the
>> Meson submodule.
>>
>> find . \( -name '*.so' -o -name '*.dll' -o -name '*.[oda]' \) -type f \
>>
>> "*.d" matches various meson files.
>>
>> (Again, apologies if this is already known; I understand this is
>> probably the worst context to report his in...)
>
> Is this running "make clean" from an old commit (i.e. as in an in-tree
> build) after having checked out the meson submodule?

The source tree is checked out at commit 9d5589bb3fee ("Merge
remote-tracking branch
'remotes/vivier2/tags/linux-user-for-5.2-pull-request' into staging",
2020-09-07).

The current working directory is the source tree.

The working directory is pristine, as in:
$ git reset --hard
$ git clean -ffdx
$ git submodule deinit --force --all

Then:

$ git submodule update --init --force
$ make clean
$ git status

> On branch master
> Your branch is up to date with 'origin/master'.
>
> Changes not staged for commit:
>   (use "git add <file>..." to update what will be committed)
>   (use "git checkout -- <file>..." to discard changes in working > directory)
>   (commit or discard the untracked or modified content in submodules)
>
>         modified:   meson (modified content)
>
> no changes added to commit (use "git add" and/or "git commit -a")

And if I change to meson:

$ cd meson
$ git status

> HEAD detached at 68ed748f84f1
> Changes not staged for commit:
>   (use "git add/rm <file>..." to update what will be committed)
>   (use "git checkout -- <file>..." to discard changes in working directory)
>
>         deleted:    test cases/d/1 simple/app.d
>         deleted:    test cases/d/1 simple/utils.d
>         deleted:    test cases/d/10 d cpp/dmain.d
>         deleted:    test cases/d/10 d cpp/libfile.d
>         deleted:    test cases/d/11 dub/test.d
>         deleted:    test cases/d/2 static library/app.d
>         deleted:    test cases/d/2 static library/libstuff.d
>         deleted:    test cases/d/3 shared library/app.d
>         deleted:    test cases/d/3 shared library/libstuff.d
>         deleted:    test cases/d/3 shared library/sub/libstuff.d
>         deleted:    test cases/d/4 library versions/lib.d
>         deleted:    test cases/d/5 mixed/app.d
>         deleted:    test cases/d/6 unittest/app.d
>         deleted:    test cases/d/6 unittest/second_unit.d
>         deleted:    test cases/d/7 multilib/app.d
>         deleted:    test cases/d/7 multilib/say1.d
>         deleted:    test cases/d/7 multilib/say2.d
>         deleted:    test cases/d/9 features/app.d
>         deleted:    test cases/d/9 features/extra.d
>
> no changes added to commit (use "git add" and/or "git commit -a")

I don't know if the "clean" target has any relevance left, in the
top-level Makefile; I've only looked at it now because the command

> 	find . \( -name '*.so' -o -name '*.dll' -o -name '*.[oda]' \) -type f \
> 		! -path ./roms/edk2/ArmPkg/Library/GccLto/liblto-aarch64.a \
> 		! -path ./roms/edk2/ArmPkg/Library/GccLto/liblto-arm.a \
> 		! -path ./roms/edk2/BaseTools/Source/Python/UPT/Dll/sqlite3.dll \
> 		-exec rm {} +

needed an update (removing the sqlite3.dll exception) for the
edk2-stable202008 update.

Thanks!
Laszlo
Paolo Bonzini Sept. 12, 2020, 6:01 a.m. UTC | #4
On 08/09/20 14:30, Laszlo Ersek wrote:
> 
> The current working directory is the source tree.
> 
> The working directory is pristine, as in:
> $ git reset --hard
> $ git clean -ffdx
> $ git submodule deinit --force --all
> 
> Then:
> 
> $ git submodule update --init --force
> $ make clean
> $ git status

I see.  We should probably rename the Makefile in the source tree, since
it is only meant to be used from the build tree.

Paolo
Laurent Vivier Sept. 16, 2020, 9:17 a.m. UTC | #5
Le 07/09/2020 à 18:12, Philippe Mathieu-Daudé a écrit :
> Since commit a56650518f5 ("configure: integrate Meson in the build

> system") we replaced many Makefile by Meson files. Adapt the

> git.orderfile script to display the new file at the same position.

> 

> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> ---

>  scripts/git.orderfile | 1 +

>  1 file changed, 1 insertion(+)

> 

> diff --git a/scripts/git.orderfile b/scripts/git.orderfile

> index 73fd818d7f3..a021afc2d51 100644

> --- a/scripts/git.orderfile

> +++ b/scripts/git.orderfile

> @@ -18,6 +18,7 @@ docs/*

>  configure

>  Makefile*

>  *.mak

> +meson.build

>  

>  # qapi schema

>  qapi/*.json

> 


Applied to my trivial-patches branch.

Thanks,
Laurent
diff mbox series

Patch

diff --git a/scripts/git.orderfile b/scripts/git.orderfile
index 73fd818d7f3..a021afc2d51 100644
--- a/scripts/git.orderfile
+++ b/scripts/git.orderfile
@@ -18,6 +18,7 @@  docs/*
 configure
 Makefile*
 *.mak
+meson.build
 
 # qapi schema
 qapi/*.json