actions/install-action
1
0
Fork 0
mirror of https://github.com/taiki-e/install-action.git synced 2026-05-10 22:50:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

codegen: Improve handling of broken versions

Browse Source
This commit is contained in:
Taiki Endo
2024-03-02 18:25:05 +09:00
parent 2e3ebeb449
commit 41e81ec8b7
2 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
manifests/cargo-rdme.json generated
View File

@@ -20,6 +20,11 @@
"1.4": { "1.4": {
"version": "1.4.2" "version": "1.4.2"
}, },
"1.4.3": {
"x86_64_windows": {
"checksum": "408ffb657c99bf4142521632306edc4ed130dd991a0cd4b62cf370c28b307daf"
}
},
"1.4.2": { "1.4.2": {
"x86_64_linux_musl": { "x86_64_linux_musl": {
"checksum": "8555ae93237e1702e79542c810657afc1ff76282c9ecd7b21aa22f5f3b3c21d2" "checksum": "8555ae93237e1702e79542c810657afc1ff76282c9ecd7b21aa22f5f3b3c21d2"

14
tools/codegen/src/main.rs
View File

@@ -62,7 +62,7 @@ fn main() -> Result<()> {
} }
releases.append(&mut r); releases.append(&mut r);
} }
let mut releases: BTreeMap<_, _> = releases let releases: BTreeMap<_, _> = releases
.iter() .iter()
.filter_map(|release| { .filter_map(|release| {
if release.prerelease { if release.prerelease {
@@ -78,9 +78,6 @@ fn main() -> Result<()> {
Some((Reverse(semver_version.ok()?), (version, release))) Some((Reverse(semver_version.ok()?), (version, release)))
}) })
.collect(); .collect();
for broken in &base_info.broken {
releases.remove(&Reverse(broken.clone()));
}
let mut crates_io_info = None; let mut crates_io_info = None;
base_info.rust_crate = base_info base_info.rust_crate = base_info
@@ -377,6 +374,9 @@ fn main() -> Result<()> {
} }
} }
} }
if base_info.broken.contains(version) {
continue;
}
if !(version.major == 0 && version.minor == 0) { if !(version.major == 0 && version.minor == 0) {
manifests.map.insert( manifests.map.insert(
Reverse(Version::omitted(version.major, Some(version.minor))), Reverse(Version::omitted(version.major, Some(version.minor))),
@@ -417,7 +417,8 @@ fn main() -> Result<()> {
{ {
bail!( bail!(
"platform list in base manifest for {package} contains {p:?}, \ "platform list in base manifest for {package} contains {p:?}, \
but result manifest doesn't contain it" but result manifest doesn't contain it; \
consider removing {p:?} from platform list in base manifest"
); );
} }
if latest_manifest.download_info.contains_key(&p) { if latest_manifest.download_info.contains_key(&p) {
@@ -435,7 +436,8 @@ fn main() -> Result<()> {
} }
bail!( bail!(
"platform list in base manifest for {package} contains {p:?}, \ "platform list in base manifest for {package} contains {p:?}, \
but latest release ({latest_version}) doesn't contain it" but latest release ({latest_version}) doesn't contain it; \
consider marking {latest_version} as broken by adding 'broken' field to base manifest"
); );
} }