cp: fix recursive copy of readonly directories

[nikolalukovic]

Attempt 2 to fix #7961

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/follow-name (passes in this run but fails in the 'main' branch)

[nikolalukovic]

Failing test is for a different util which I didn't touch.
Since I'm using the created value only in a single case when calling copy_attributes maybe it would be a good idea to turn the function parameter into an Option?

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)

[nikolalukovic]

@sylvestre can you please take a look? failing test is unrelated to these changes