[swift]: add 6.2.4 and 6.3 support#2045
Conversation
|
@mattgodbolt @partouf – not a high priority, but before possibly merging this, I'd like your feedback on the approach to support patch releases that I've started to implement here and is outlined in more detail in compiler-explorer/compiler-explorer#8393. Thanks in advance! |
|
@jamieQ looks like this broke the tests ... happy to look once those give it a clean bill of health |
|
oh! no apologies; please update and merge latest to get the fix for the tests.... |
|
This installation seems ok: will opine on the other PR later |
|
@mattgodbolt per your feedback on the issue I linked, it sounds to me like we shouldn't do what I was proposing and should just have each compiler version be saved "forever". Given that existing compilers won't be overwritten, it seems my thinking about how to reduce some of the tasks involved for updating things wouldn't actually work. So, I think it might be better to revert this and I'll re-land a change that just keeps things aligned with how they were previously (each compiler version gets its own directory matching its version number). Edit: or is it too late to change that if this has been merged? |
|
(also, thank you for your reply!) |
|
We can change :-) sorry I didn't think through your PR before I merged it: feel free to send a new one and I'll update |
@mattgodbolt Thanks! Here it is: #2054 |
Adds support for the 6.3 and 6.2.4 swift compilers. Depends on: compiler-explorer/infra#2045, compiler-explorer/infra#2054
2 participants
Note: This PR introduces a slight change in the organization of the newly-added release compilers. Specifically it extracts the new compilers into a new directory that is intended to hold any subsequent patch release compilers for a given minor version release (6.2 has had 4 patch releases thus far). The intent is to achieve the sort of thing laid out in compiler-explorer/compiler-explorer#8393.