Skip to content
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.

Sign up for GitHub

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 3 commits into from
Nov 27, 2024
Merged

Add meeting notes 2024-11-27 #2388

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3f69899
Add community/minutes/2024-11-27.md
github-actions[bot] Nov 27, 2024
17afd3f
Update community/minutes/2024-11-27.md
github-actions[bot] Nov 27, 2024
fe25196
Update community/minutes/2024-11-27.md
beckermr Nov 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
61 changes: 61 additions & 0 deletions community/minutes/2024-11-27.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
---
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 |
| | | | |
| | | | |
beckermr marked this conversation as resolved.
Show resolved Hide resolved
| 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`
Copy link
Member

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?

Copy link
Member

@jjerphan jjerphan Nov 28, 2024 •
edited
Loading

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.

Copy link
Member

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.

Copy link
Member

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

Copy link
Member

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.

Copy link
Member

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 super prudent and assume that ABI-compatibility depends on platforms.

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

- [ ] 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

- [ ]