Skip to content

Commit

Permalink
Drop Python 3.8 artifacts from the release (#340)
Browse files Browse the repository at this point in the history
We need more disk space for 3.13t and this seems like the most straight-forward way to get it?
  • Loading branch information
zanieb authored Oct 7, 2024
1 parent 075ad3d commit 82e9015
Showing 1 changed file with 52 additions and 30 deletions.
82 changes: 52 additions & 30 deletions src/github.rs
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ async fn fetch_artifact(
repo: &str,
artifact: WorkflowListArtifact,
) -> Result<bytes::Bytes> {
println!("downloading {}", artifact.name);
println!("downloading artifact {}", artifact.name);

let res = client
.actions()
Expand Down Expand Up @@ -107,6 +107,8 @@ pub async fn command_fetch_release_distributions(args: &ArgMatches) -> Result<()
)
.build()?;

let release_version_range = pep440_rs::VersionSpecifier::from_str(">=3.9")?;

let workflows = client.workflows(org, repo);

let mut workflow_names = HashMap::new();
Expand Down Expand Up @@ -208,49 +210,69 @@ pub async fn command_fetch_release_distributions(args: &ArgMatches) -> Result<()

let name = zf.name().to_string();

let parts = name.split('-').collect::<Vec<_>>();

if parts[0] != "cpython" {
println!("ignoring {} not a cpython artifact", name);
continue;
};

let python_version = pep440_rs::Version::from_str(parts[1])?;
if !release_version_range.contains(&python_version) {
println!(
"{} not in release version range {}",
name, release_version_range
);
continue;
}

// Iterate over `RELEASE_TRIPLES` in reverse-order to ensure that if any triple is a
// substring of another, the longest match is used.
if let Some((triple, release)) =
let Some((triple, release)) =
RELEASE_TRIPLES.iter().rev().find_map(|(triple, release)| {
if name.contains(triple) {
Some((triple, release))
} else {
None
}
})
{
let stripped_name = if let Some(s) = name.strip_suffix(".tar.zst") {
s
} else {
println!("{} not a .tar.zst artifact", name);
continue;
};
else {
println!(
"ignoring {} does not match any registered release triples",
name
);
continue;
};

let stripped_name = &stripped_name[0..stripped_name.len() - "-YYYYMMDDTHHMM".len()];
let stripped_name = if let Some(s) = name.strip_suffix(".tar.zst") {
s
} else {
println!("ignoring {} not a .tar.zst artifact", name);
continue;
};

let triple_start = stripped_name
.find(triple)
.expect("validated triple presence above");
let stripped_name = &stripped_name[0..stripped_name.len() - "-YYYYMMDDTHHMM".len()];

let build_suffix = &stripped_name[triple_start + triple.len() + 1..];
let triple_start = stripped_name
.find(triple)
.expect("validated triple presence above");

if !release.suffixes(None).any(|suffix| build_suffix == suffix) {
println!("{} not a release artifact for triple", name);
continue;
}
let build_suffix = &stripped_name[triple_start + triple.len() + 1..];

let dest_path = dest_dir.join(&name);
let mut buf = vec![];
zf.read_to_end(&mut buf)?;
std::fs::write(&dest_path, &buf)?;
if !release.suffixes(None).any(|suffix| build_suffix == suffix) {
println!("ignoring {} not a release artifact for triple", name);
continue;
}

println!("releasing {}", name);
let dest_path = dest_dir.join(&name);
let mut buf = vec![];
zf.read_to_end(&mut buf)?;
std::fs::write(&dest_path, &buf)?;

if build_suffix == release.install_only_suffix {
install_paths.push(dest_path);
}
} else {
println!("{} does not match any registered release triples", name);
println!("prepared {} for release", name);

if build_suffix == release.install_only_suffix {
install_paths.push(dest_path);
}
}
}
Expand All @@ -271,7 +293,7 @@ pub async fn command_fetch_release_distributions(args: &ArgMatches) -> Result<()
let dest_path = produce_install_only(path)?;

println!(
"releasing {}",
"prepared {} for release",
dest_path
.file_name()
.expect("should have file name")
Expand All @@ -290,7 +312,7 @@ pub async fn command_fetch_release_distributions(args: &ArgMatches) -> Result<()
let dest_path = produce_install_only_stripped(&dest_path, &llvm_dir)?;

println!(
"releasing {}",
"prepared {} for release",
dest_path
.file_name()
.expect("should have file name")
Expand Down

0 comments on commit 82e9015

Please sign in to comment.