aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/repository/writable_array_repository.rs
diff options
context:
space:
mode:
Diffstat (limited to 'crates/shirabe/src/repository/writable_array_repository.rs')
-rw-r--r--crates/shirabe/src/repository/writable_array_repository.rs24
1 files changed, 13 insertions, 11 deletions
diff --git a/crates/shirabe/src/repository/writable_array_repository.rs b/crates/shirabe/src/repository/writable_array_repository.rs
index f5db2bd9..bd685652 100644
--- a/crates/shirabe/src/repository/writable_array_repository.rs
+++ b/crates/shirabe/src/repository/writable_array_repository.rs
@@ -13,7 +13,9 @@ use shirabe_semver::constraint::AnyConstraint;
#[derive(Debug)]
pub struct WritableArrayRepository {
inner: ArrayRepository,
- pub(crate) dev_package_names: Vec<String>,
+ // RefCell so that FilesystemRepository::initialize can stay `&self` (late-bound
+ // initialization out of shared contexts such as getRepoName).
+ pub(crate) dev_package_names: std::cell::RefCell<Vec<String>>,
dev_mode: Option<bool>,
}
@@ -21,7 +23,7 @@ impl WritableArrayRepository {
pub fn new(packages: Vec<crate::package::PackageInterfaceHandle>) -> anyhow::Result<Self> {
Ok(Self {
inner: ArrayRepository::new(packages)?,
- dev_package_names: Vec::new(),
+ dev_package_names: std::cell::RefCell::new(Vec::new()),
dev_mode: None,
})
}
@@ -36,12 +38,12 @@ impl WritableArrayRepository {
self.inner.base_count()
}
- pub fn set_dev_package_names(&mut self, dev_package_names: Vec<String>) {
- self.dev_package_names = dev_package_names;
+ pub fn set_dev_package_names(&self, dev_package_names: Vec<String>) {
+ *self.dev_package_names.borrow_mut() = dev_package_names;
}
- pub fn get_dev_package_names(&self) -> &Vec<String> {
- &self.dev_package_names
+ pub fn get_dev_package_names(&self) -> Vec<String> {
+ self.dev_package_names.borrow().clone()
}
pub fn write(
@@ -66,7 +68,7 @@ impl WritableArrayRepository {
}
pub fn add_package(
- &mut self,
+ &self,
package: crate::package::PackageInterfaceHandle,
) -> anyhow::Result<()> {
self.inner.add_package(package)
@@ -84,7 +86,7 @@ impl WritableArrayRepository {
Ok(())
}
- pub fn initialize(&mut self) -> anyhow::Result<()> {
+ pub fn initialize(&self) -> anyhow::Result<()> {
self.inner.initialize();
Ok(())
}
@@ -123,8 +125,8 @@ impl WritableArrayRepository {
self.inner.get_packages()
}
- pub fn get_repo_name(&self) -> String {
- self.inner.get_repo_name()
+ pub fn get_repo_name(&self) -> anyhow::Result<String> {
+ RepositoryInterface::get_repo_name(&self.inner)
}
}
@@ -188,7 +190,7 @@ impl RepositoryInterface for WritableArrayRepository {
self.inner.get_providers(package_name)
}
- fn get_repo_name(&self) -> String {
+ fn get_repo_name(&self) -> anyhow::Result<String> {
self.inner.get_repo_name()
}