Aaron Puchert changed bug 1223378
What Removed Added
Status NEW IN_PROGRESS
Assignee screening-team-bugs@suse.de aaronpuchert@alice-dsl.net

Comment # 7 on bug 1223378 from Aaron Puchert
(In reply to Michael Pujos from comment #6)
> I had a go at fixing it, updating it in the process to the latest update
> v2024.04.0+735.tar.gz.

Sorry, I should have said that I was working on it.
(https://build.opensuse.org/package/show/home:aaronpuchert:branches:devel:languages:R:released/rstudio)

Maybe you can have a look. I solved some parts differently, but I copied your
approach towards Leap. (With a minor adjustment: copy_options is already
available with Boost 1.75.)

> Due to the complexity of this package and the time it takes to compile, it
> was not exactly a piece of cake and required a few hours of digging.

Yes, unfortunately we had to patch out some stuff.

> Interestingly, even git rstudio does not compile with boost 1.85: it
> required a trivial 1 line patch.

Upstream seems to target Boost 1.83 for now, the relevant change is only in
Boost 1.85.
(https://github.com/boostorg/function/commit/af8e66d03b4366c7039a138306470567c2243130)

> The new version starts but I am not a rstudio user nor familiar with R, so
> it would be great if someone familiar with it could test it a bit and report
> before I submit it to the devel project:
> 
> https://build.opensuse.org/package/show/home:bobbie424242:branches:openSUSE:
> Factory/rstudio

I don't use RStudio a lot, but most of the stuff I've been doing in the past
continues to work.

> Note that this version does not compile on Leap 15.5 because it has older
> boost 1.66 and it would require conditionally reverting
> https://github.com/rstudio/rstudio/commit/
> 251e441e1034950db8a054652dffb6348194f231, which I may do later.

Interestingly, SLE has Boost 1.75 since SP3
(https://build.opensuse.org/package/show/SUSE:SLE-15-SP3:GA/boost.175), but
instead of libboost_*-devel it provides libboost_*-devel-impl. And it conflicts
with Boost 1.66, which we transitively require via e.g. soci-devel.

So I think patching it as you did is our only option.

I think we shouldn't do

BuildRequires:  openSUSE-release

Because it might cause frequent rebuilds (at least in the devel project) and it
doesn't sound right. RStudio being a part of the distribution shouldn't depend
on it. I'd propose to just patch this out. There is nothing SUSE-specific at
the moment in the CMake files, and if there is it might not work for Tumbleweed
anyway.


You are receiving this mail because: