cp: Add tests for symbolic link case.
ClosedPublic
Actions

Authored by des on Dec 13 2023, 11:05 PM.

Details

Reviewers

kevans
allanjude

Group Reviewers

Klara
tests

Commits

rGd3a8e9b43b4c: cp: Add tests for symbolic link case.

Summary

MFC after: 1 week
Sponsored by: Klara, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 54939
Build 51828: arc lint + arc unit

Event Timeline

des created this revision.Dec 13 2023, 11:05 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 13 2023, 11:05 PM

des requested review of this revision.Dec 13 2023, 11:05 PM

Harbormaster completed remote builds in B54939: Diff 131410.Dec 13 2023, 11:05 PM

It didn't occur to me until after I'd already committed D43052 that we didn't have tests for -s either (the basic_symlink case just checks that a normal cp does nothing if the destination is already a link to the source, or vice versa).

kevans accepted this revision as: Klara, kevans.Dec 13 2023, 11:09 PM

This revision is now accepted and ready to land.Dec 13 2023, 11:09 PM

reviewed-by: allanjude

Closed by commit rGd3a8e9b43b4c: cp: Add tests for symbolic link case. (authored by des). · Explain WhyDec 13 2023, 11:34 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGd3a8e9b43b4c: cp: Add tests for symbolic link case..

des mentioned this in rGd514cadacc50: cp: Add tests for hard link case..Jan 17 2024, 6:28 PM

des mentioned this in rG006017419bea: cp: Add tests for hard link case..

Revision Contents
Changeset List

Path

Size

bin/

cp/

tests/

cp_test.sh

31 lines

Diff 131410

View Options

cp: Add tests for symbolic link case.ClosedPublicActions