Re: [PATCH v2 4/4] make: replace make by $(MAKE)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Lianbo,

lijiang <lijiang@xxxxxxxxxx> writes:

> This Message Is From an External Sender 
> This message came from outside your organization. 
> Thank you for the reply, Sven.
>
> On Thu, Feb 10, 2022 at 2:37 PM Sven Schnelle <svens@xxxxxxxxxxxxx> wrote:
>
>  Hi,
>
>  lijiang <lijiang@xxxxxxxxxx> writes:
>
>  > This Message Is From an External Sender 
>  > This message came from outside your organization. 
>  > Hi, Sven and Kazu
>  >
>  > The patchset really saves compilation time for crash build and the following three patches look good to me(need a
>  minor
>  > adjustment as Kazu mentioned).
>  > Thank you for the work, Sven.
>  >
>  > [PATCH v2 1/4] make: set --no-print-directory once
>  > [PATCH v2 3/4] make: use -C instead of (cd x; make)
>  > [PATCH v2 4/4] make: replace make by $(MAKE)
>  >
>  > Given that the gdb patch has specific rules,  Kazu(or me) can help to
>  > pack them into one patch with Sven's signature, if you have no objections.
>
>  Sorry, i was busy with other stuff. If you could fix that patching issue
>  (i haven't really looked into this), that would be great!
>
>  
> No worries. I will discuss this one with Kazu later, once ready, we will try to merge it.

I've just looked through the commits, and it looks like this patch was
never adjusted/merged. Do you think you can change that one and push it,
or should i try to do it and re-submit?

Thanks
Sven

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility
Contribution Guidelines: https://github.com/crash-utility/crash/wiki




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux