-
-
Notifications
You must be signed in to change notification settings - Fork 284
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add meeting notes 2024-11-27 #2388
Merged
Merged
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,59 @@ | ||
| --- | ||
| tags: [meeting-notes] | ||
| title: '2024-11-27' | ||
| --- | ||
| # conda-forge core meeting 2024-11-27 | ||
|
|
||
| Add new agenda items under the `Your __new__() agenda items` heading | ||
|
|
||
| - [Zoom link](https://zoom.us/j/9138593505?pwd=SWh3dE1IK05LV01Qa0FJZ1ZpMzJLZz09) | ||
| - [What time is the meeting in my time zone](https://dateful.com/convert/utc?t=2pm) | ||
| - [Previous meetings](https://conda-forge.org/community/minutes/) | ||
|
|
||
| ## Attendees | ||
|
|
||
| | Name | Initials | GitHub ID | Affiliation | | ||
| | ----------------------- | -------- | --------------- | --------------------------- | | ||
| | | | | | | ||
| | Filipe Fernandes | FF | ocefpaf | conda-forge | | ||
| | Uwe Korn | UK | xhochy | conda-forge/QuantCo | | ||
| | Wolf Vollprecht | WV | wolfv | prefix.dev | | ||
| | Marius van Niekerk | MvN | mariusvniekerk | conda-forge/Voltron Data | | ||
| | Matthew Becker | MRB | beckermr | cf | | ||
|
|
||
| X people total | ||
|
|
||
| ### Standing items | ||
|
|
||
| - [ ] | ||
|
|
||
| ### From previous meeting(s) | ||
|
|
||
| - [ ] | ||
|
|
||
| ### Active votes | ||
|
|
||
| - [ ] | ||
|
|
||
| ### Your __new__() agenda items | ||
|
|
||
| - [x] (MRB) packages.conda-forge.org as canonical repodata location? | ||
| - idea is to redirect this to anaconda.org for now | ||
| - that could change in the future | ||
| - [x] (MRB) `noarch: python` w/ `python_min` migration started | ||
| - this is a big one ~13k feedstocks | ||
| - doing one per bot run for now to fix errors | ||
| - [x] (WV) multi-output cache for v1 recipes | ||
| - [x] Already available with `--experimental` | ||
| - [x] Wrote up a (proto)-CEP https://github.com/conda/ceps/pull/102 | ||
| - [x] (UK) `-fno-omit-frame-pointer` | ||
| - [ ] fedora did this | ||
| - [ ] might slow things down a very tiny bit | ||
|
|
||
| ### Pushed to next meeting | ||
|
|
||
| - [ ] (ME) Composite action to build installers (Miniconda, Miniforge, etc.) | ||
|
|
||
| ### CFEPs | ||
|
|
||
| - [ ] | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was the outcome of this discussion? I remember that @jjerphan and colleagues were interested in this. AFAIU it's an ABI break though, so we'd probably have to queue that up for whenever we rebuild the world?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For such a change, I would be more than prudent and assume that ABI-compatibility depends on platforms.
If they were to be activated again, I guess the best would be to have a conda-forge reboot or at least a batch rebuild for an entire set of packages.
See this part of Fedora Project Wiki for more information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it actually an ABI break to activate frame pointers? That sounds unexpected to me: you can compile any code with or without frame pointers, and still link to the same existing third-party or system libs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xhochy was going to make a PR and see what people said
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but could it in practice alter some calling conventions?
I am not knowledgeable or experienced enough to be able to tell; I think it might be worth contacting people who re-activated them on RPM-based or DEB-based distributions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I'm confused about whether this would change the ABI. My understanding is that a specific register is always filled, independent of whether the function uses it or not. I don't think this should be a world-rebuilt.
Let's move this discussion to the PR over here: conda-forge/clang-compiler-activation-feedstock#151