Make `pijul_repository::max_files()` infallible

finchie

Jul 22, 2025, 6:39 AM

EFY2CITFEAIGNPBPNXY3RV4LONYYSXKBNWPVPHKTCHCFK2OBQ5AAC

Dependencies

• [2] OGJFEWHU Fixing missing dependencies on partial clones
• [3] CCLLB7OI Upgrading to Sanakirja 0.15 + version bump
• [4] ABQDWHNG Migrate from `pijul::repository` to `pijul-repository`
• [5] EUZFFJSO Updating Pijul with the latest changes in Libpijul
• [6] YN63NUZO Sanakirja 1.0
• [7] H72JG6HL Init feature; specific .ignore configs
• [8] SXEYMYF7 Fixing the bad changes in history (unfortunately, by rebooting).
• [9] L4JXJHWX pijul/*: reorganize imports and remove extern crate
• [10] JL4WKA5P Implement the Sanakirja concurrency model in a cross-process way
• [11] DDJO7X2P Remove dependency on `num_cpus`
• [12] 76PCXGML Pushing to, and pulling from the local repository
• [13] TYAKEAJL A better estimate of the maximum number of open patches we can keep (Unix-only at the moment)

Change contents

• replacement in pijul-repository/src/lib.rs at line 9
  [3.23]→[3.23:39](∅→∅)
  − use log::debug;
  [3.23]

  [3.39]
  + use log::{debug, warn};
• replacement in pijul-repository/src/lib.rs at line 33
  [3.14]→[3.0:47](∅→∅)
  − pub fn max_files() -> std::io::Result<usize> {
  [3.14]

  [3.44]
  + pub fn max_files() -> usize {
• replacement in pijul-repository/src/lib.rs at line 35
  [3.122]→[3.48:128](∅→∅)
  − (n as usize / (2 * std::thread::available_parallelism()?.get())).max(1)
  [3.122]

  [3.174]
  + let available_parallelism = match std::thread::available_parallelism() {

  + Ok(available_parallelism) => available_parallelism.get(),

  + Err(error) => {

  + warn!("Failed to get available parallelism: {error}");

  + 1

  + }

  + };

  +

  + (n as usize / (2 * available_parallelism)).max(1)
• edit in pijul-repository/src/lib.rs at line 47
  [3.206]

  [3.206]
  +
• replacement in pijul-repository/src/lib.rs at line 49
  [3.241]→[3.129:139](∅→∅)
  − Ok(n)
  [3.241]

  [3.247]
  + n
• replacement in pijul-repository/src/lib.rs at line 53
  [3.268]→[3.140:187](∅→∅),[3.187]→[3.187:197](∅→∅)
  − pub fn max_files() -> std::io::Result<usize> {

  − Ok(1)
  [3.268]

  [3.304]
  + pub fn max_files() -> usize {

  + 1
• replacement in pijul-repository/src/lib.rs at line 110
  [3.421]→[3.198:228](∅→∅)
  − max_files()?,
  [3.421]

  [3.469]
  + max_files(),
• replacement in pijul-repository/src/lib.rs at line 148
  [3.594]→[3.229:263](∅→∅)
  − max_files()?,
  [3.594]

  [3.627]
  + max_files(),
• replacement in pijul-remote/src/local.rs at line 104
  [2.658]→[2.658:702](∅→∅)
  − pijul_repository::max_files()?,
  [2.658]

  [2.702]
  + pijul_repository::max_files(),
• replacement in pijul-remote/src/local.rs at line 219
  [3.1284]→[3.767:811](∅→∅)
  − pijul_repository::max_files()?,
  [3.1284]

  [3.1328]
  + pijul_repository::max_files(),
• replacement in pijul-remote/src/lib.rs at line 1041
  [3.764]→[3.570:622](∅→∅)
  − pijul_repository::max_files()?,
  [3.764]

  [3.816]
  + pijul_repository::max_files(),
• replacement in pijul-remote/src/lib.rs at line 1123
  [3.952]→[3.623:675](∅→∅)
  − pijul_repository::max_files()?,
  [3.952]

  [3.1004]
  + pijul_repository::max_files(),