[R-pkg-devel] new package version fails on r-devel fedora clang

[Tony Wilkes]

Hi Kevin, CRAN, and the rest,

Thank you for your answer, Kevin, I appreciate it!

If there is an issue in the compiler CRAN uses, it seems reasonable that CRAN fixes that.
I also don't know how to work around a bug in a compiler. How would you even do that?

Literally the only concrete thing Brian Ripley said to me was "Taking 5h CPU time to (fail to) install is unreasonable -- resubmitting without addressing the problem more so." Ignoring the fact that I did indeed try to address the problem, the only think I can do (and have been doing), is trying to reduce the compiled size of the C/C++ code, to reduce installation time.

But if there is a bug in CRAN's compiler, re-submitting will probably just end up getting another fail, regardless of how small the package becomes. That would be a waste of CRAN's time and resources. I appreciate CRAN, and so I don't like wasting their time and resources.

@BrianRipley/@CRAN: I'm very willing to submit a package with a significantly smaller compiled size (and thus hopefully faster installation). But I ask again: please be so kind to at least check your clang compiler. Asking me to fix a problem caused by CRAN's own compiler seems unreasonable. If you say the compiler is good or has been fixed, I'll re-submit my package with the reduced size & installation time. Does that sound fair?

Thanks in advance for your response!

Kind regards,

Tony

________________________________
Van: Kevin Ushey <kevinushey using gmail.com>
Verzonden: maandag 6 oktober 2025 22:25
Aan: Tony Wilkes <tony_a_wilkes using outlook.com>
CC: List r-package-devel <r-package-devel using r-project.org>
Onderwerp: Re: [R-pkg-devel] new package version fails on r-devel fedora clang

The error log at
https://www.r-project.org/nosvn/R.check/r-devel-linux-x86_64-fedora-clang/broadcast-00install.html<https://www.r-project.org/nosvn/R.check/r-devel-linux-x86_64-fedora-clang/broadcast-00install.html>
makes it pretty clear that there's a bug in the compiler being used on
that CRAN machine, but that might still not absolve you of the
responsibility of working around the issue.

At least, this line:

5. Running pass "jump-threading" on function
"_Z16rcpp_bcRel_dec_dP7SEXPRECS0_S0_S0_S0_S0_S0_lid"

indicates that the bug is happening while compiling the
'cpp_bcRel_dec_d' function.

Is there another way of expressing the computation you're doing in
that function that might avoid hitting this issue? From a brief scan
of the code, it's hard to give concrete advice because of the heavy
use of macros within that code.

Best,
Kevin

On Sun, Oct 5, 2025 at 10:35 AM Tony Wilkes <tony_a_wilkes using outlook.com> wrote:
>
> Hello everyone,
>
> I have fixed the 'broadcast' package (updated to version 0.1.5.1), but there is still an installation error on r-devel-linux-x86_64-fedora-clang: https://www.r-project.org/nosvn/R.check/r-devel-linux-x86_64-fedora-clang/broadcast-00check.html<https://www.r-project.org/nosvn/R.check/r-devel-linux-x86_64-fedora-clang/broadcast-00check.html>.
> Either there is still an issue with broadcast, or there is actually an issue with the compiler CRAN uses for fedora.
> If there is still an issue with broadcast: What exactly is the issue? The install.log is not clear to me.
> If there is an issue on the side of CRAN: May I respectfully request CRAN to double-check their clang compiler and re-run the checks for broadcast 0.1.5.1?
>
> Thanks in advance!
>
> Kind regards,
>
> Tony
> ________________________________
> Van: Tony Wilkes <tony_a_wilkes using outlook.com>
> Verzonden: vrijdag 3 oktober 2025 18:39
> Aan: Dirk Eddelbuettel <edd using debian.org>; CRAN <cran-submissions using r-project.org>
> Onderwerp: Re: [R-pkg-devel] new package version fails on r-devel fedora clang
>
> Hi Dirk and CRAN,
>
>
> @Dirk: thanks so much for the trouble!
>
> I actually think I know what went wrong. My code is not wrong per-sé.
> I created an addition if()else() statement in rcpp_bc_b, but left the MACRO - which itself has many if-else statements - the same. And each of those "if" leads to a whole set of for-loops, thus causing the compiled code to become much larger than necessary.
>
> It's very easy to fix.
> But it does feel like a rookie mistake; sorry for the trouble!
>
> @CRAN: Sorry! I was wrong, and your system is probably correct. If it's OK with you, I'll send an updated version of 'broadcast' tonight. That should fix the error in the Fedora clang checks.
>
> Kind regards,
>
> Tony
>
>
>
>
> ________________________________
> Van: Dirk Eddelbuettel <edd using debian.org>
> Verzonden: vrijdag 3 oktober 2025 17:50
> Aan: Dirk Eddelbuettel <edd using debian.org>
> CC: Tony Wilkes <tony_a_wilkes using outlook.com>
> Onderwerp: Re: [R-pkg-devel] new package version fails on r-devel fedora clang
>
>
> (off list now)
>
> On 3 October 2025 at 10:08, Dirk Eddelbuettel wrote:
> | For another test I ensured I also used -O3 and -D_FORTIFY_SOURCE=3 (switching
> | from the Debian defaults of 2). There are a number of other flags that
> | differ, and they may matter.  It still appears to build (if slowly, seems
> | like there is a fair bit of complex C++ in there). [ Minutes later ] It seems
> | to hang at compilation of rcpp_bc_b.cpp.
>
> It eventually made it, without error, but it took a long time.
>
> Hth, Dirk
>
> --
> dirk.eddelbuettel.com | @eddelbuettel | edd using debian.org
>
>         [[alternative HTML version deleted]]
>
> ______________________________________________
> R-package-devel using r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-package-devel<https://stat.ethz.ch/mailman/listinfo/r-package-devel>

	[[alternative HTML version deleted]]