DPP icon indicating copy to clipboard operation
DPP copied to clipboard

docs: Altered minimum gcc version for Coro, fixed docs issues

Open Jaskowicz1 opened this issue 1 year ago • 7 comments

This PR raises the minimum gcc/g++ version for Coro from 11 to 13.

This PR also corrects set_emoji (too much text for a @brief, moved most of it to @warning and @note) and set_disabled (previously stated that both true and false would disable a component).

Documentation change checklist

  • [x] My documentation changes follow the docs style guide and any code examples follow the coding style guide.
  • [x] I tested that my change works before raising the PR (via running doxygen, and testing examples).
  • [x] I have not moved any existing pages or changed any existing URLs without strong justification as to why.
  • [x] I have not generated content using AI or a desktop utility such as grammarly.

Jaskowicz1 avatar Jun 18 '24 11:06 Jaskowicz1

Deploy Preview for dpp-dev ready!

Name Link
Latest commit 15d5993d6f8eba915931edd42f0993d5ed0c6b6d
Latest deploy log https://app.netlify.com/sites/dpp-dev/deploys/667734280e72b90008e0d24e
Deploy Preview https://deploy-preview-1173--dpp-dev.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Jun 18 '24 11:06 netlify[bot]

are you sure 13 is the minimum? ive been using it on v12 in seven spells and beholder just fine, and on my local machine...

braindigitalis avatar Jun 19 '24 07:06 braindigitalis

are you sure 13 is the minimum? ive been using it on v12 in seven spells and beholder just fine, and on my local machine...

Not entirely sure to be honest with you. I was under the impression it was gcc13 as minimum.

If you're running coro on 12 then I'll change it to gcc12!

Jaskowicz1 avatar Jun 19 '24 08:06 Jaskowicz1

It's for ARM that you need 13

For x86 11 and 12 work but have bugs with lambdas (we already warn about this one in docs)

Mishura4 avatar Jun 19 '24 12:06 Mishura4

For x86 11 and 12 work but have bugs with lambdas (we already warn about this one in docs)

https://discord.com/channels/825407338755653642/825411707521728512/1252545524717064235 https://discord.com/channels/825407338755653642/825411707521728512/1225743126602317834 is it?

Seems like 11 and 12 have issues on these machines (both cases are debian 12 though, so maybe it's a debian thing).

Jaskowicz1 avatar Jun 19 '24 13:06 Jaskowicz1

It's for ARM that you need 13

Mishura4 avatar Jun 19 '24 15:06 Mishura4

So, to now update this, it seems it's only gcc12.3 or higher and then a specific version of gcc11 that works.

From talks, it's best to just tell people to use gcc13 or higher. This answers your question about it working on your end @braindigitalis

Jaskowicz1 avatar Jun 19 '24 16:06 Jaskowicz1