caisin/typst
1
0
Fork 0
mirror of https://github.com/typst/typst synced 2025-07-27 14:27:56 +08:00
Code Issues Packages Projects Releases Wiki Activity

first attempt

Browse Source
This commit is contained in:
Neven Villani 2025-06-11 22:32:10 +02:00
parent a5ef75a9c2
commit 91ec946283
1 changed files with 36 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
crates/typst-kit/src/package.rs
View File

@ -87,13 +87,9 @@ impl PackageStorage {
) -> PackageResult<PathBuf> {
let subdir = format!("{}/{}/{}", spec.namespace, spec.name, spec.version);
if let Some(packages_dir) = &self.package_path {
let dir = packages_dir.join(&subdir);
if dir.exists() {
return Ok(dir);
}
}
// QUESTION: is it sometimes the case that both package_path
// and package_cache_path are Some? Is it a problem if we change
// this order?
if let Some(cache_dir) = &self.package_cache_path {
let dir = cache_dir.join(&subdir);
if dir.exists() {
@ -102,14 +98,39 @@ impl PackageStorage {
// Download from network if it doesn't exist yet.
if spec.namespace == DEFAULT_NAMESPACE {
self.download_package(spec, cache_dir, progress)?;
if dir.exists() {
return Ok(dir);
}
return self.download_package(spec, cache_dir, progress);
}
}
Err(PackageError::NotFound(spec.clone()))
// In all paths from now, we either succeed or fail with a NotFound.
let not_found = |msg| Err(PackageError::NotFound(spec.clone(), msg));
let Some(packages_dir) = &self.package_path else {
// QUESTION: when is this code path hit?
return not_found(None);
};
// We try the fast path
let dir = packages_dir.join(&subdir);
if dir.exists() {
return Ok(dir);
}
// Fast path failed. Instead we try to diagnose the issue by going
// down the subdirectories of the expected path one at a time.
let namespace_dir = packages_dir.join(format!("{}", spec.namespace));
if !namespace_dir.exists() {
return not_found(Some(eco_format!("did not find namespace '{}' while searching at {} for", spec.namespace, namespace_dir.display())));
}
let package_dir = namespace_dir.join(format!("{}", spec.name));
if !package_dir.exists() {
return not_found(Some(eco_format!("did not find package '{}' while searching at {} for", spec.name, package_dir.display())));
}
let version_dir = package_dir.join(format!("{}", spec.version));
if !version_dir.exists() {
return not_found(Some(eco_format!("did not find version {} while searching at {} for", spec.version, version_dir.display())));
}
// We'll never hit this path as long as subdir is indeed namespace/name/version.
// What's Typst's policy on `unreachable`?
unreachable!()
}
/// Tries to determine the latest version of a package.
@ -171,7 +192,7 @@ impl PackageStorage {
spec: &PackageSpec,
cache_dir: &Path,
progress: &mut dyn Progress,
) -> PackageResult<()> {
) -> PackageResult<PathBuf> {
assert_eq!(spec.namespace, DEFAULT_NAMESPACE);
let url = format!(
@ -235,8 +256,8 @@ impl PackageStorage {
// broken packages still occur even with the rename safeguard, we might
// consider more complex solutions like file locking or checksums.
match fs::rename(&tempdir, &package_dir) {
Ok(()) => Ok(()),
Err(err) if err.kind() == io::ErrorKind::DirectoryNotEmpty => Ok(()),
Ok(()) => Ok(package_dir),
Err(err) if err.kind() == io::ErrorKind::DirectoryNotEmpty => Ok(package_dir),
Err(err) => Err(error("failed to move downloaded package directory", err)),
}
}