Anirban Halder

The ``since`` option in uptime also seems to be deprecated, its not listed in https://www.gnu.org/software/coreutils/manual/html_node/uptime-invocation.html . I want to pick this up, if anyone is not already working on it.

Changes since last push : - Changed the logic to use ``` let md_res = get_metadata_with_deref_opt(target.p_buf.as_path(), path.must_dereference); let md = md_res.or(target.p_buf.symlink_metadata()); apply_style_based_on_metadata(path, md.as_ref(),ok(), ls_colors, style_manager, &name) ``` - Added test

Changes since last push: - Using ``at.symlink_file`` for symlink creation - Switched to regex matching for extracting colors from stdout - Squashed the commit into an older comment formatting commit...

I was wondering if i should clean up the commit history a bit more ? Maybe only leave two commits one for the main change and another for the test...

Changes summary since last review. 1. Testing using file with dynamically created binary utmp records. Not testing for macos , since i don't have access to make to figure out...

Changes since last push: The test ``test_uptime::test_uptime_for_file_without_utmpx_records`` has been disabled for ``freebsd`` , our ``uucore::utmpx`` module uses ``utmpxname()`` sys call to set the file from which we iterate through, freebsd...

The failing CI tests are unrelated to my PR.

@sylvestre does this require any more changes ?

Changes since last push: - Removed quick_error in favor of thiserror - Added some documentation on the added dependencies. The clippy error seems to be unrelated to my pr

I am the author of the ``test_cp_debug_reflink_auto_sparse_always_non_sparse_file_with_long_zero_sequence`` test, that specific test wasn't created with taking cow filesystems like btrfs into account as I simply didn't have access to a non...