Bug 211644 – Remove empty directories from from svn.webkit.org repository

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[David Kilzer (:ddkilzer)]

Remove empty directories from from svn.webkit.org repository.

I updated my Subversion checkout of WebKit after about 18 months, and found sound empty directories:

I used this command to find and remove the empty directories:

$ for D in `find -d . -type d -and -not -name .svn | grep -v '/.svn'`; do if [ -z "`ls -A $D 2>&1`" ]; then echo ">>>> $D"; svn rm $D; fi; done 
[...]
$ svn stat .
D       LayoutTests/fast/events/touch/ios/iphone7
D       LayoutTests/fast/text-autosizing/ios/ipad
D       LayoutTests/fast/viewport/ios/ipad
D       LayoutTests/imported/mozilla/css-animations
D       LayoutTests/imported/mozilla/css-transitions
D       LayoutTests/imported/w3c/web-platform-tests/web-animations/interfaces/AnimationEffectTiming
D       LayoutTests/legacy-animation-engine
D       LayoutTests/media/controls/ipad
D       LayoutTests/media/modern-media-controls/media-documents/ipad
D       LayoutTests/media/modern-media-controls/pip-support/ipad
D       LayoutTests/platform/gtk/legacy-animation-engine
D       LayoutTests/platform/ios/compositing/filters
D       LayoutTests/platform/ios/legacy-animation-engine
D       LayoutTests/platform/ios/media/modern-media-controls
D       LayoutTests/platform/ios-wk1/legacy-animation-engine
D       LayoutTests/platform/ios-wk2/legacy-animation-engine
D       LayoutTests/platform/iphone-7/tiled-drawing
D       LayoutTests/platform/mac/legacy-animation-engine
D       LayoutTests/platform/mac-wk1/compositing/plugins
D       LayoutTests/platform/mac-wk1/imported/w3c/web-platform-tests/fetch/content-type
D       LayoutTests/platform/win/legacy-animation-engine
D       LayoutTests/platform/wincairo/legacy-animation-engine
D       LayoutTests/platform/wpe/legacy-animation-engine
D       LayoutTests/tiled-drawing/ios
D       Source/ThirdParty/ANGLE/include/vulkan
D       Source/ThirdParty/ANGLE/src/android_system_settings
D       Source/ThirdParty/ANGLE/third_party/glmark2
D       Source/ThirdParty/ANGLE/tools/flex-bison
D       Source/WebCore/Modules/webgpu/cocoa
D       Source/WebCore/page/scrolling/ios
D       Source/WebCore/platform/graphics/metal
D       Source/WebKit/UIProcess/API/win
D       Tools/BuildSlaveSupport/gtk
D       Tools/WebKitTestRunner/WebKitTestRunnerApp/en.lproj
D       Tools/flatpak/files
D       Tools/flatpak/patches

Any concerns about removing these directories?  (I will revert Tools/WebKitTestRunner/WebKitTestRunnerApp/en.lproj as In think that's used.)

[David Kilzer (:ddkilzer)]

BTW, I can't post a patch (only a ChangeLog) because svn-create-patch doesn't currently support creating a patch that deletes directories, and git-svn basically ignores empty directories when checking out or updating a repository.

[David Kilzer (:ddkilzer)]

Created attachment 398905 [details]
Patch v1

[EWS Watchlist]

Note that there are important steps to take when updating ANGLE. See http://trac.webkit.org/wiki/UpdatingANGLE

[David Kilzer (:ddkilzer)]

Committed r261429: <https://trac.webkit.org/changeset/261429>

[Radar WebKit Bug Importer]

<rdar://problem/63044462>

[David Kilzer (:ddkilzer)]

Note to self or to future empty directory deleters:  The `find -d` switch does a depth-first search, which means the `find` command only needs to be run once instead of multiple times (see Bug 189081) to remove all the empty directories.