This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v2] Fix install-strip for cross-compilation
- From: "Hannes Domani via gdb-patches" <gdb-patches at sourceware dot org>
- To: Gdb-patches <gdb-patches at sourceware dot org>
- Date: Wed, 1 Jan 2020 20:59:54 +0000 (UTC)
- Subject: Re: [PATCH v2] Fix install-strip for cross-compilation
- References: <20191231184222.34267-1-ssbssa.ref@yahoo.de> <20191231184222.34267-1-ssbssa@yahoo.de> <64c719e2-df88-6cd8-3f5b-246f0f7b7d58@simark.ca> <1134374112.8470290.1577819135686@mail.yahoo.com> <d0973170-34fd-ff20-f6ef-8218a663eb51@simark.ca>
- Reply-to: Hannes Domani <ssbssa at yahoo dot de>
Am Dienstag, 31. Dezember 2019, 21:12:21 MEZ hat Simon Marchi <simark@simark.ca> Folgendes geschrieben:
> On 2019-12-31 2:05 p.m., Hannes Domani via gdb-patches wrote:
> > Am Dienstag, 31. Dezember 2019, 19:50:47 MEZ hat Simon Marchi <simark@simark.ca> Folgendes geschrieben:
> >
> >> On 2019-12-31 1:42 p.m., Hannes Domani via gdb-patches wrote:
> >>
> >>> The variable INSTALL_PROGRAM_ENV sets up STRIPPROG for the cross-compiler.
> >>>
> >>> If this is not done, the host 'strip' is used, and fails:
> >>>
> >>> /bin/sh /c/src/repos/binutils-gdb.git/install-sh -c -s gdb.exe \
> >>> /gdb/gdb64-git/bin/$transformed_name.exe
> >>> strip.exe:C:/gdb/gdb64-git/bin/_inst.33599_: file format not recognized
> >>>
> >>> With this change, it's fine:
> >>>
> >>> STRIPPROG='x86_64-w64-mingw32-strip' \
> >>> /bin/sh /c/src/repos/binutils-gdb.git/install-sh -c -s gdb.exe \
> >>> /gdb/gdb64-git/bin/$transformed_name.exe
> >>
> >>
> >> Thanks for updating the change log, this LGTM. I believe you don't have write access
> >> to the repo, is that right? Would you like me to push the patch on your behalf, or
> >> would you like to get a write access (useful if you plan on sending patches regularly)?
> >
> > Yes, I don't have write access, and I do plan to send more patches in the future.
> >
> > With write access, is there more to it than just push the patch?
>
> Ok, if you do not yet have a Sourceware account, please fill this form to get one:
>
> https://sourceware.org/cgi-bin/pdw/ps_form.cgi
>
> You can mention me as your sponsor. If you already have an account, you need to contact
> the Sourceware overseers mailing list so they can adjust the permissions to give you write
> access to binutils-gdb.
>
> To push a patch, it's indeed just a matter of pushing the git commit. However, you need
> to insert the ChangeLog entries in the actual ChangeLog files and amend your commit before
> pushing. Also, it's good to triple check that there is only what you intend to push in the
> branch you are about to push.
>
> Once you have write access, you can add yourself to the MAINTAINERS file, in the "write after
> approval" section and then send a "FYI" patch to this list, that gives you a chance to practice.
>
> I won't push this patch, I'll let you do it once you have your account.
>
> Also, I see that you already have a copyright assignment files with the FSF, that's perfect.
I've now added myself to the MAINTAINERS file, and committed this patch.
Regards
Hannes Domani