Re: Improve Solaris mudflap support (PR libmudflap/49550)

2011-07-06 Thread Frank Ch. Eigler
Hi, Rainer - It addresses a couple of testsuite failures: [...] where the registration of __iob has been done automatically by the compiler. I avoid this problem by not registering stdin, stdout, and stderr separately on Solaris. OK. * Some tests were failing while calling

Re: Improve Solaris mudflap support (PR libmudflap/49550)

2011-07-06 Thread Frank Ch. Eigler
Hi, Rainer - If so, I'd rather do it by adding some MFWRAP_OS_SPEC to avoid having to duplicate the whole spec in the Solaris config headers. Why would solaris have to duplicate MFWRAP_SPEC if mmap64 is added to the default gcc.c one? I assumed that you wanted to keep the

Re: More mudflap fixes for Solaris 11

2011-07-13 Thread Frank Ch. Eigler
Hi, Rainer - When testing libmudflap on Solaris 8, 9, and 10 with GNU ld, I found a couple of testsuite failures: [...] Ok for mainline? Yes, thank you! - FChE

Re: Define [CD]TORS_SECTION_ASM_OP on Solaris/x86 with Sun ld

2011-07-13 Thread Frank Ch. Eigler
Hi, Rainer - On Mon, Jul 11, 2011 at 06:34:27PM +0200, Rainer Orth wrote: [...] On the other hand, there's the question why tree-mudflap.c tries to create a constructor with a non-default priority on a platform with SUPPORTS_INIT_PRIORITY == 0 or at all [...] For the at all part, I believe

Re: [PATCH] PR c/53702: Fix -Wunused-local-typedefs with nested functions

2012-06-21 Thread Frank Ch. Eigler
Hi - This is OK. You should have write-after-approval access. Overseers, please add user meadori to group gcc. Done. - FChE

Re: Unreviewed build/libgcc, mudflap patches

2011-06-27 Thread Frank Ch. Eigler
Hi - On Mon, Jun 27, 2011 at 01:32:30PM +0200, Rainer Orth wrote: [...] Support libmudflap on Solaris (PR libmudflap/38738) http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01446.html Looks good to me, thanks for taking a look at the area. - FChE

Re: Unreviewed build/libgcc, mudflap patches

2011-06-27 Thread Frank Ch. Eigler
Hi - http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01446.html Looks good to me, thanks for taking a look at the area. Could you also comment on the open questions in the submission so mudflap support on Solaris can be improved further? Sure. Transcribing: One generic issue came

Re: GCC wwwdocs move to git done

2019-10-08 Thread Frank Ch. Eigler
Hi - Thanks - good job with moving this to git! > Note 1: someone with the right access needs to create the symlink > /sourceware/git/gcc-wwwdocs.git -> > /sourceware/projects/gcc-home/wwwdocs.git (and anything else needed for > anonymous git access to that repository). Done. - FChE

[wwwdocs] tweak for sourceware account request alias

2024-01-11 Thread Frank Ch. Eigler
diff --git a/htdocs/gitwrite.html b/htdocs/gitwrite.html index 0c146aba44d2..c89cdb8fb2ef 100644 --- a/htdocs/gitwrite.html +++ b/htdocs/gitwrite.html @@ -36,7 +36,7 @@ be sponsored by an existing maintainer (someone with "write after approval" is not sufficient). If you already have an

Re: [PATCH] contrib: use sphinx-build from a venv

2022-07-26 Thread Frank Ch. Eigler via Gcc-patches
Hi - > CCing overseers and Frank. > Can you please help me with that? > > Can please a maintainer install the package from pip? > > Something like: > > virtualenv /home/gcc/venv && /home/gcc/venv/bin/pip install Sphinx > > or a similar location? The gccadmin team can do this kind of thing

Re: [PATCH] contrib: use sphinx-build from a venv

2022-07-26 Thread Frank Ch. Eigler via Gcc-patches
Hi - > > The gccadmin team can do this kind of thing without overseer/root > > privileges, or indeed so can any local shell-privileged user. > > Yeah, I said I didn't want to install it that way without overseer > approval, as pip won't keep the packages up to date the way dnf > installations

Re: Rust frontend patches v2

2022-08-25 Thread Frank Ch. Eigler via Gcc-patches
Hi - > 12K 0004-gccrs-Add-link-cases-testsuite.patch > 356K0005-gccrs-Add-general-compilation-test-cases.patch > 132K0006-gccrs-Add-execution-test-cases.patch > 4.0K0007-gccrs-Add-gcc-check-target-check-rust.patch > 656K