Re: [Crash-utility] Couple of update requests

On Sat, Jun 12, 2021 at 5:52 AM Luc Park-Chouinard <lucchouina(a)gmail.com&gt; wrote: > > Thanks for updating the page. > I amended the fix to use a separate option: > https://github.com/crash-utility/crash/pull/86/commits/308e279a71b1a81512... Also for the amended change, Acked-by: Kazuhito Hagio <k-hagio-ab(a)nec.com&gt; Thanks, Kazu > > On Thu, Jun 10, 2021 at 8:20 PM HAGIO KAZUHITO(萩尾　一仁) <k-hagio-ab(a)nec.com&gt; wrote: >> >> Hi Luc, >> >> -----Original Message----- >> > When you can. >> > (let me know if I need to go through a different channel for these changes) >> > >> > >> > 1. Updates to the extension page (https://crash-utility.github.io/extensions.html) >> > >> > * change email associated with eppic extension from >> > >> > * lchouinard(a)s2sys.com <mailto:lchouinard@s2sys.com> >> > >> > * to: >> > >> > * lucchouina(a)gmail.com <mailto:lucchouina@gmail.com> >> > >> > * change eppic code pointer from : >> > >> > * http://code.google.com/p/eppic >> > >> > * to: >> > >> > * https://github.com/lucchouina/eppic >> >> Updated. >> https://github.com/crash-utility/crash-utility.github.io/commit/9eb28df82... >> >> If there is error/lack of fix, please let me know. >> >> > >> > 2. A (tiny) pull request >> > I know email patches are preferred, but this is a trivial (one-liner) change for the Eppic extension >> > build handling. >> > https://github.com/crash-utility/crash/pull/86 >> > >> >> OK, the main reason why email patches are preferred is to make the patches >> and discussion open to more watchers here. In that sense, you sent this >> email, it almost meets the purpose. >> >> And, I thought that introducing something like $(EPPIC_GIT_OPTION) may be >> also good, but I'm ok with either. >> >> Acked-by: Kazuhito Hagio <k-hagio-ab(a)nec.com&gt; >> >> Thanks, >> Kazu >> >> >> >> -- >> Crash-utility mailing list >> Crash-utility(a)redhat.com >> https://listman.redhat.com/mailman/listinfo/crash-utility >> > > > -- > > - Luc > -- > Crash-utility mailing list > Crash-utility(a)redhat.com > https://listman.redhat.com/mailman/listinfo/crash-utility