summaryrefslogtreecommitdiffstats
path: root/lib/core
AgeCommit message (Collapse)Author
2020-05-31handlebars: 2 -> 3Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-05-31semver: 0.9.0 -> 0.10.0Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-05-31itertools: 0.8.0 -> 0.9.0Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-04-29Replace repository with new sourcehut repoMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-04-26Clippy Fix: let_underscore_lockMatthias Beyer
Use _variable instead of _ as a variable name for lock. Expl.: https://rust-lang.github.io/rust-clippy/master/index.html#let_underscore_lock Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-24Merge branch 'update-toml-query' into masterMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-24Fix: Return actual length of bufferMatthias Beyer
The function has to return the number of bytes in the buffer, so return the length of the buffer here. Still, the contents are ignored and not written to any destination. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-24Add blackhole as default cli optionMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-24Update toml-query: 0.9.2 -> 0.10.0Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-24Revert "Move to toml-query from master branch"Matthias Beyer
This reverts commit f7fadc118f2de7e84cef8c6d4103296289564f74. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-24Fix clippy warning: next() instead of nth(0)Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-08Move to toml-query from master branchMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-08Move libimagerror to thiserrorMatthias Beyer
In this patch, the error output (tracing) algorithm was changed, but should essentially print the same information. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-08Replace failure with anyhow in complete codebaseMatthias Beyer
This patch was scripted with sed -i 's/use failure::Error/use anyhow::Error/' $(rg "use failure::Error" -l) sed -i 's/use failure::Fallible as /use anyhow::/' $(rg "use failure::Fallible" -l) sed -i 's/failure/anyhow/' $(rg "failure *=" -l) sed -i 's/format_err!/anyhow!/' $(rg "format_err!" -l) sed -i 's/use failure::ResultExt/use anyhow::Context/' $(rg "use failure::ResultExt" -l) sed -i 's/err_msg/anyhow!/' $(rg "use failure::err_msg" -l) sed -i 's/^anyhow\ *=.*$/anyhow = "1"/' $(rg "anyhow * =" -l) sed -i 's/^anyhow_derive.*//' $(rg "anyhow_derive" -l) sed -i 's/extern crate failure/extern crate anyhow/' $(rg "extern crate failure" -l) sed -i 's/.*extern crate anyhow_derive.*//' $(rg "anyhow_derive" -l) Some manual changes were added as well, so this patch was not completely scripted, but mostly. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-06Remove unused str error handling moduleMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-06Remove unused io error handling codeMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-06Remove unused ExitUnwrap traitMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-06Remove unused iter extensionsMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-01Add iterator extension for reporting from iteratorMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-03-01Allow more fine-granular control over IO behaviour in runtimeMatthias Beyer
This patch removes the --ignore-ids option and adds --pipe-input / -I and --pipe-output / -O instead. The behaviour of -IO is the same as --ignore-ids was before, but this change gives the user the option to enable only one of both (input or output, or both). Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-02-28Remove module-based loggingMatthias Beyer
This patch removes the module-based logging mechanism. This mechanism was introduced so that the configuration file is able to define which imag module logs what. This was a fine-granular setting, but most users won't actually touch it. It was more of a debugging-thing for developers. I rarely used it during development and it only introduced more headaches. It also was not updated (as in the configuration file missed a few modules, others where not removed despite the module did not exist anymore). All in all, it was rather unmaintained and just "too much". Thus, remove this thing completely. Developers know how to use grep. This also automatically fixes bugs where `imag --debug command` did not output any debugging log, but was expected to. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-02-28Fix: If global loglevel allowes record to be logged, log itMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-02-21Add test whether Store::is_borrowed() works as expectedMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-01-03Add Store::is_borrowed(StoreId)Matthias Beyer
Add a function to Store to check whether a StoreId is borrowed. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2020-01-01Update copyright years to 2015-2020Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-12-25Fix: Remove Box::leak() call in runtime application abstractionMatthias Beyer
This patch removes the leakage in the runtime application abstraction. Also fix formatting. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-12-23Remove unnecessary clone() callMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-12-23Remove DebugResult and DebugOption helpersMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-12-23Replace map_dbg() and map_dbg_str() with context() and inspect() callsMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-12-23Replace map_dbg_err() calls with context() callsMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-12-22Fix: Ignore broken pipe when reporting idsMatthias Beyer
This fixes a bug where a broken pipe in Runtime::report_touched() resulted in an Err(_) raised up to main() which then reported this. But as report_touched() should ignore a broken pipe (because the program will exit anyways shortly after the call), we can safely ignore this error. This also results in `ExitCode` removed from the function signature, which pushes us forward to the removal of custom error-handling implementations! Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-11-09Add Runtime::input_is_pipe()Matthias Beyer
We need this because we might want to check whether we can get the STDIN handle from the runtime or not. In some (rare) cases, the runtime "thinks" that STDIN is a stream of IDs, but it is not. So the client can check whether stdin is a pipe (thus, whether Runtime thinks that there's a stream of IDs) and easily work around that by taking ::std::io::stdin() itself. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-27Add extension trait for iterator over Result<Entry>Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-27Add iterator extensions for store-id touched reportingMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Revert "Add extension traits for handling Result<Option<T>, E> conversion"Matthias Beyer
This reverts commit d810b63886f4006d01bca894982bbc867616f37c. As resiter v0.4.0 does provide this extension now, we don't need it in the codebase anymore.
2019-10-26Add extension traits for handling Result<Option<T>, E> conversionMatthias Beyer
This extension traits help transforming Result<Option<T>, E> to Result<T, E> or transforming an iterator over the former type to an iterator over the latter type. Should be moved to resiter of course, but we need to implement this first. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Implement Error for ExitCodeMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Add Entry::set_content()Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Add ImagApplication trait for CLI completion + single-binary supportLeon Schuermann
To re-add the imag-binary CLI completion, as well as to enable building a single imag containing all of the subcommands, this commit introduces an ImagApplication trait to be implemented by all imag binary crates. The binary crates will be converted to libraries, with an additional binary target. This enables standalone and single binary builds. On its own, this commit doesn't do much, but rather it paves the way to dynamically interacting with the imag uis/clis using a library-crate interface. Signed-off-by: Leon Schuermann <leon@is.currently.online>
2019-10-26Add some debug outputMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Fix test: Check whether in cache, then get, then check againMatthias Beyer
After moving an entry, the entry should _not_ be in the cache. This is just a decision that has to be made, whether we cache the moved entry or not. I decided to not cache because it is results in less code. After that check, we get the entry from the backend and then we can check whether the entry is in the cache, which is then should be. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Add test: Test moving entry with writing to it before and after movingMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Fix: Testing backend bug where an entry was not properly rewrittenMatthias Beyer
When moving an entry, what we did is copying the entry inside the backend abstraction (the hashmap) from one key to another. But as the entry itself does also encode its location, we actually have to rewrite the entire entry. This patch fixes this. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-26Fix: Renaming entries should also remove the old entryMatthias Beyer
This is a bugfix for a bug in Store::move_entry_by_id(): The old entry was not removed after the new one was created. The bug is a bit subtle. The issue was, that the internal cache held open a reference to the old entry (as StoreEntry object) and when that object got drop()ed, the contents of the entry were written to disk, which resulted in the old entry being recreated. This patch rewrites the function to behave well. The most critical part is that we do not check whether the old entry is borrowed with `hsmap.get()` but rather `hsmap.remove()`. The trick here is that the `StoreEntry` object is dropped in the moment the check is done, clearing the cached object and flushing it to the backend (the disk). After that, we continue doing the filesystem operation and the cache is clean. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-13Fix: --override-config argument can be passed multiple timesMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-10-07Update dependency: ansi_term: 0.11.0 -> 0.12Matthias Beyer
2019-10-07Update dependency: handlebars: 1.1.0 -> 2Matthias Beyer
2019-10-07Update dependency: env_logger: 0.6.1 -> 0.7Matthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
2019-09-01Add Tests for Entries::{find_by_id_substr, find_by_id_startswith}Matthias Beyer
2019-09-01Add Entries::{find_by_id_substr, find_by_id_startswith}Matthias Beyer
This patch adds functions to the Entries type which can be used for filtering by id, either with `contains()` or `starts_with()`. This is useful for the end-user functionality where the user specifies the ID of an entry only partially. The implementation still iterates over all entries. This could be improved, of course, by implementing a `find`-like function on `Store` directly. But for now, this is good enough. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
generated by cgit v1.2.3 (git 2.25.1) at 2024-06-08 11:41:21 +0000