FreeBSD source tree https://cgit-dev.freebsd.org/src/atom?h=releng%2F12.4 2022-11-02T15:51:05Z 2022-11-02T15:51:05Z Dag-Erling Smørgrav des@FreeBSD.org 2022-09-09T13:08:43Z urn:sha1:73ab1c87c208dbb0609be572eb31591353d71e07 Differential Revision: https://reviews.freebsd.org/D36505 MFC after: 1 month (cherry picked from commit 497cdf9673ec53d59634bb78862cd8dea3d5c155) sh: when loading profile, skip obvious scratch files. Differential Revision: https://reviews.freebsd.org/D36856 (cherry picked from commit d3890a547d201d1f5043e51baa406ac9d5d3a60a) sh: when loading profile, read only .sh files. Reviewers: jilles, eugen_grosbein.net, cy Subscribers: imp Differential Revision: https://reviews.freebsd.org/D37034 (cherry picked from commit 8d7221ca2dd5e0b6c633761b314cffed82ddca72) (cherry picked from commit 4ea49af3afe127c424a1b843617225fd437b51d1) Approved by: re (gjb) 2022-04-25T00:02:30Z Cameron Katri me@cameronkatri.com 2022-02-23T18:55:13Z urn:sha1:19413ce66cc02d676e0778eb9519284e47d7d8e1 According to POSIX, cp should allow the `-P` flag to work whether `-R` is specified or not. Currently, the `-P` option only works along with `-R`. PR: 199466 Reviewed by: kevans (cherry picked from commit 97e13037915c22162f199461f56951793d669f57) 2022-04-25T00:02:16Z Kyle Evans kevans@FreeBSD.org 2022-01-27T18:02:17Z urn:sha1:1d9f60b0512802806d3dea3c5fcc6a0d8780259f The traversal was previously not properly honoring -H/-L/-P. Notably, we should not have been resolving symlinks encountered during traversal when either -H or -P are specified. Sponsored by: Klara, Inc. Reviewed by: bapt (cherry picked from commit 33ad990ce974be50abdc25427b0f365699d83a34) 2022-04-24T03:49:10Z Kyle Evans kevans@FreeBSD.org 2022-04-06T01:40:53Z urn:sha1:a605ca25ef681452e25622a44917cbb5033eaae4 The correct logic is a lot simpler than the previous iteration. We record the base fts_name to avoid having to worry about whether we needed the root symlink name or not (as applicable), then we can simply shift all of that logic to after path translation to make it less fragile. If we're copying to DNE, then we'll have swapped out the NULL root_stat pointer and then attempted to recurse on it. The previously nonexistent directory shouldn't exist at all in the new structure, so just back out from that tree entirely and move on. The tests have been amended to indicate our expectations better with subdirectory recursion. If we copy A to A/B, then we expect to copy everything from A/B/* into A/B/A/B, with exception to the A that we create in A/B. Reviewed by: bapt Sponsored by: Klara, Inc. (cherry picked from commit f00f8b4fbd268a212687984e44daa3e0d0a16b87) 2022-04-24T03:48:56Z Kyle Evans kevans@FreeBSD.org 2022-01-27T18:02:17Z urn:sha1:b57954717ddfe52be9e18591d302c61e50db0aa6 As noted in the PR, cp -R has some surprising behavior. Typically, when you `cp -R foo bar` where both foo and bar exist, foo is cleanly copied to foo/bar. When you `cp -R foo foo` (where foo clearly exists), cp(1) goes a little off the rails as it creates foo/foo, then discovers that and creates foo/foo/foo, so on and so forth, until it eventually fails. POSIX doesn't seem to disallow this behavior, but it isn't very useful. GNU cp(1) will detect the recursion and squash it, but emit a message in the process that it has done so. This change seemingly follows the GNU behavior, but it currently doesn't warn about the situation -- the author feels that the final product is about what one might expect from doing this and thus, doesn't need a warning. The author doesn't feel strongly about this. PR: 235438 Reviewed by: bapt Sponsored by: Klara, Inc. (cherry picked from commit 848263aad129c8f9de75b58a5ab9a010611b75ac) 2022-04-18T14:44:30Z Stefan Eßer se@FreeBSD.org 2022-04-18T14:44:30Z urn:sha1:879a6b7b628060a85a0f1787aa30ab6efbf1e0c1 A last minute clean-up of the Makefile erroneously removed the definitions required to link against sbin/mount/getmntops.c which defines the iovec handling functions used for the unmounted device case (that has been removed from -CURRENT and 13-STABLE). Put back the definitions to make the build with MOUNT_CHAR_DEVS defined succeed. 2022-04-18T12:00:33Z Stefan Eßer se@FreeBSD.org 2022-04-18T11:56:38Z urn:sha1:7dfb8f15e2692a16c4e1bba4b8cb5c5d85f5cfe1 When resolving conflicts due to the unmergewd commit that removed the code previously made conditional on MOUNT_CHAR_DEVS, I failed to see that MOUNT needed to be defined in the Makefile and that the include of mntopts.h was still required for that case. Fix the build by adding back the missing definitions to the Makefile. 2022-04-18T11:04:04Z Stefan Eßer se@FreeBSD.org 2022-02-10T20:09:34Z urn:sha1:741b90dc69ee9e9f3bbd2b0548ea214030c4ea34 The df command provides a -l option to exclude all non-local file systems and a -t option with a (positive or negative) list of file system types to display. This commit adds support for a combination of -l and -t. If both are specified, the parameter list of the -t option is applied on top of the selection of öocal file systems (independently of the order of the -l and -t options). E.g., "df -t noprocfs,sysfs -l" will select all local file systems except those of type procfs and sysfs. (cherry picked from commit f0fd4a32c4f7ab5eb008ab08a1a13cb20d4b30d2) (cherry picked from commit c888fc2d97ee3e25269ca8218f80769bca930705) 2022-04-18T08:44:23Z Mateusz Guzik mjg@FreeBSD.org 2021-07-20T17:00:50Z urn:sha1:647f70e6bf4213f3a892277fcafe2c36d81aa3a5 Found with cc --analyze Sponsored by: Rubicon Communications, LLC ("Netgate") (cherry picked from commit 570b31f61009fb60591cf5fcc26f0f5c12ff5752) (cherry picked from commit ede6e97b828dec1bb9aee4e69c2f8b4e437ebef7) 2022-04-18T08:41:10Z Fernando Apesteguía fernape@FreeBSD.org 2020-10-05T13:39:37Z urn:sha1:ca5f7b7a320ea259d6a0b6c498f6c73f0a5f92b3 * Add EXAMPLES section with four simple examples. * Simplify -H flag description. This makes easy to see the difference between this flag and -h * While here, fix .Tn deprecated macro. Approved by: manpages (bcr@) Differential Revision: https://reviews.freebsd.org/D26662 (cherry picked from commit ad3d2e7e4a18575f825b65a19e49720a513d2343)