Skip to content

Add recipe for r-sweater#4298

Closed
rgaiacs wants to merge 1 commit intoemscripten-forge:emscripten-3xfrom
rgaiacs:r-sweater
Closed

Add recipe for r-sweater#4298
rgaiacs wants to merge 1 commit intoemscripten-forge:emscripten-3xfrom
rgaiacs:r-sweater

Conversation

@rgaiacs
Copy link
Copy Markdown
Contributor

@rgaiacs rgaiacs commented Jan 26, 2026

During a conversation on Zulip, @martinRenou recommended that I sent a pull request to this repository.

A recipe for r-sweater was contributed to conda-forge, see https://github.com/conda-forge/r-sweater-feedstock/, but part of sweater is written in C++, see https://github.com/gesistsa/sweater/tree/c0790cb312503160d3239991e49e9e7b56f101d8/src.

The Adding packages section of the documentation is missing a subsection about R packages as mentioned in #934. I can use the experience on this pull request a first version of the documentation.

@IsabelParedes
Copy link
Copy Markdown
Member

Hi @rgaiacs ! Thank you for your PR. If it's okay with you, could you reopen this PR and target the emscripten-4x branch instead of main? We are migrating all of the packages to that branch and I am no longer keeping the R packages on the main branch up to date.
And you are correct, the documentation needs to be updated as well.

- R/library/${{ name[2:] }}/libs/${{ name[2:] }}.so

about:
home: https://github.com/gesistsa/sweater
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
home: https://github.com/gesistsa/sweater
homepage: https://github.com/gesistsa/sweater

@IsabelParedes IsabelParedes added the R R language packages label Jan 26, 2026
@rgaiacs
Copy link
Copy Markdown
Contributor Author

rgaiacs commented Jan 27, 2026

I'm closing this in favour of #4305 given the previous comment. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R R language packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants