Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 1 | // Copyright 2017 Google Inc. All rights reserved. |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | // you may not use this file except in compliance with the License. |
| 5 | // You may obtain a copy of the License at |
| 6 | // |
| 7 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | // See the License for the specific language governing permissions and |
| 13 | // limitations under the License. |
| 14 | |
| 15 | package build |
| 16 | |
| 17 | import ( |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 18 | "bytes" |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 19 | "fmt" |
| 20 | "io/ioutil" |
| 21 | "os" |
| 22 | "path/filepath" |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 23 | "sort" |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 24 | "strings" |
Nan Zhang | 17f2767 | 2018-12-12 16:01:49 -0800 | [diff] [blame] | 25 | |
| 26 | "android/soong/ui/metrics" |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 27 | ) |
| 28 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 29 | // Given a series of glob patterns, remove matching files and directories from the filesystem. |
| 30 | // For example, "malware*" would remove all files and directories in the current directory that begin with "malware". |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 31 | func removeGlobs(ctx Context, globs ...string) { |
| 32 | for _, glob := range globs { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 33 | // Find files and directories that match this glob pattern. |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 34 | files, err := filepath.Glob(glob) |
| 35 | if err != nil { |
| 36 | // Only possible error is ErrBadPattern |
| 37 | panic(fmt.Errorf("%q: %s", glob, err)) |
| 38 | } |
| 39 | |
| 40 | for _, file := range files { |
| 41 | err = os.RemoveAll(file) |
| 42 | if err != nil { |
| 43 | ctx.Fatalf("Failed to remove file %q: %v", file, err) |
| 44 | } |
| 45 | } |
| 46 | } |
| 47 | } |
| 48 | |
| 49 | // Remove everything under the out directory. Don't remove the out directory |
| 50 | // itself in case it's a symlink. |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 51 | func clean(ctx Context, config Config) { |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 52 | removeGlobs(ctx, filepath.Join(config.OutDir(), "*")) |
| 53 | ctx.Println("Entire build directory removed.") |
| 54 | } |
| 55 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 56 | // Remove everything in the data directory. |
| 57 | func dataClean(ctx Context, config Config) { |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 58 | removeGlobs(ctx, filepath.Join(config.ProductOut(), "data", "*")) |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 59 | ctx.Println("Entire data directory removed.") |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 60 | } |
| 61 | |
| 62 | // installClean deletes all of the installed files -- the intent is to remove |
| 63 | // files that may no longer be installed, either because the user previously |
| 64 | // installed them, or they were previously installed by default but no longer |
| 65 | // are. |
| 66 | // |
| 67 | // This is faster than a full clean, since we're not deleting the |
| 68 | // intermediates. Instead of recompiling, we can just copy the results. |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 69 | func installClean(ctx Context, config Config) { |
| 70 | dataClean(ctx, config) |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 71 | |
| 72 | if hostCrossOutPath := config.hostCrossOut(); hostCrossOutPath != "" { |
| 73 | hostCrossOut := func(path string) string { |
| 74 | return filepath.Join(hostCrossOutPath, path) |
| 75 | } |
| 76 | removeGlobs(ctx, |
| 77 | hostCrossOut("bin"), |
| 78 | hostCrossOut("coverage"), |
| 79 | hostCrossOut("lib*"), |
| 80 | hostCrossOut("nativetest*")) |
| 81 | } |
| 82 | |
| 83 | hostOutPath := config.HostOut() |
| 84 | hostOut := func(path string) string { |
| 85 | return filepath.Join(hostOutPath, path) |
| 86 | } |
| 87 | |
Colin Cross | 3e6f67a | 2020-10-09 19:11:22 -0700 | [diff] [blame] | 88 | hostCommonOut := func(path string) string { |
| 89 | return filepath.Join(config.hostOutRoot(), "common", path) |
| 90 | } |
| 91 | |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 92 | productOutPath := config.ProductOut() |
| 93 | productOut := func(path string) string { |
| 94 | return filepath.Join(productOutPath, path) |
| 95 | } |
| 96 | |
| 97 | // Host bin, frameworks, and lib* are intentionally omitted, since |
| 98 | // otherwise we'd have to rebuild any generated files created with |
| 99 | // those tools. |
| 100 | removeGlobs(ctx, |
Roland Levillain | e5f9ee5 | 2019-09-11 14:50:08 +0100 | [diff] [blame] | 101 | hostOut("apex"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 102 | hostOut("obj/NOTICE_FILES"), |
| 103 | hostOut("obj/PACKAGING"), |
| 104 | hostOut("coverage"), |
| 105 | hostOut("cts"), |
| 106 | hostOut("nativetest*"), |
| 107 | hostOut("sdk"), |
| 108 | hostOut("sdk_addon"), |
| 109 | hostOut("testcases"), |
| 110 | hostOut("vts"), |
Dan Shi | 984c129 | 2020-03-18 22:42:00 -0700 | [diff] [blame] | 111 | hostOut("vts10"), |
Dan Shi | 53f1a19 | 2019-11-26 10:02:53 -0800 | [diff] [blame] | 112 | hostOut("vts-core"), |
Colin Cross | 3e6f67a | 2020-10-09 19:11:22 -0700 | [diff] [blame] | 113 | hostCommonOut("obj/PACKAGING"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 114 | productOut("*.img"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 115 | productOut("*.zip"), |
Dan Willemsen | a18660d | 2017-06-01 14:23:36 -0700 | [diff] [blame] | 116 | productOut("android-info.txt"), |
Daniel Norman | b8e7f81 | 2020-05-07 16:39:36 -0700 | [diff] [blame] | 117 | productOut("misc_info.txt"), |
Steven Moreland | 0aabb11 | 2019-08-26 11:31:33 -0700 | [diff] [blame] | 118 | productOut("apex"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 119 | productOut("kernel"), |
Yo Chiang | d813f12 | 2021-01-22 00:16:47 +0800 | [diff] [blame^] | 120 | productOut("kernel-*"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 121 | productOut("data"), |
| 122 | productOut("skin"), |
| 123 | productOut("obj/NOTICE_FILES"), |
| 124 | productOut("obj/PACKAGING"), |
Tom Cherry | 7803a01 | 2018-08-08 13:24:32 -0700 | [diff] [blame] | 125 | productOut("ramdisk"), |
Bowgo Tsai | 5145c2c | 2019-10-08 18:12:37 +0800 | [diff] [blame] | 126 | productOut("debug_ramdisk"), |
Will McVicker | 4cee625 | 2020-03-19 11:57:11 -0700 | [diff] [blame] | 127 | productOut("vendor-ramdisk"), |
| 128 | productOut("vendor-ramdisk-debug.cpio.gz"), |
| 129 | productOut("vendor_debug_ramdisk"), |
Bowgo Tsai | 5145c2c | 2019-10-08 18:12:37 +0800 | [diff] [blame] | 130 | productOut("test_harness_ramdisk"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 131 | productOut("recovery"), |
| 132 | productOut("root"), |
| 133 | productOut("system"), |
| 134 | productOut("system_other"), |
| 135 | productOut("vendor"), |
Colin Cross | ce4c7cd | 2020-10-14 11:29:16 -0700 | [diff] [blame] | 136 | productOut("vendor_dlkm"), |
Jaekyun Seok | f6307cc | 2018-05-16 12:25:41 +0900 | [diff] [blame] | 137 | productOut("product"), |
Justin Yun | d5f6c82 | 2019-06-25 16:47:17 +0900 | [diff] [blame] | 138 | productOut("system_ext"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 139 | productOut("oem"), |
| 140 | productOut("obj/FAKE"), |
| 141 | productOut("breakpad"), |
| 142 | productOut("cache"), |
| 143 | productOut("coverage"), |
| 144 | productOut("installer"), |
| 145 | productOut("odm"), |
Colin Cross | ce4c7cd | 2020-10-14 11:29:16 -0700 | [diff] [blame] | 146 | productOut("odm_dlkm"), |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 147 | productOut("sysloader"), |
| 148 | productOut("testcases")) |
| 149 | } |
| 150 | |
| 151 | // Since products and build variants (unfortunately) shared the same |
| 152 | // PRODUCT_OUT staging directory, things can get out of sync if different |
| 153 | // build configurations are built in the same tree. This function will |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 154 | // notice when the configuration has changed and call installClean to |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 155 | // remove the files necessary to keep things consistent. |
| 156 | func installCleanIfNecessary(ctx Context, config Config) { |
| 157 | configFile := config.DevicePreviousProductConfig() |
| 158 | prefix := "PREVIOUS_BUILD_CONFIG := " |
| 159 | suffix := "\n" |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 160 | currentConfig := prefix + config.TargetProduct() + "-" + config.TargetBuildVariant() + suffix |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 161 | |
Dan Willemsen | e0879fc | 2017-08-04 15:06:27 -0700 | [diff] [blame] | 162 | ensureDirectoriesExist(ctx, filepath.Dir(configFile)) |
| 163 | |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 164 | writeConfig := func() { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 165 | err := ioutil.WriteFile(configFile, []byte(currentConfig), 0666) // a+rw |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 166 | if err != nil { |
| 167 | ctx.Fatalln("Failed to write product config:", err) |
| 168 | } |
| 169 | } |
| 170 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 171 | previousConfigBytes, err := ioutil.ReadFile(configFile) |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 172 | if err != nil { |
| 173 | if os.IsNotExist(err) { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 174 | // Just write the new config file, no old config file to worry about. |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 175 | writeConfig() |
| 176 | return |
| 177 | } else { |
| 178 | ctx.Fatalln("Failed to read previous product config:", err) |
| 179 | } |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 180 | } |
| 181 | |
| 182 | previousConfig := string(previousConfigBytes) |
| 183 | if previousConfig == currentConfig { |
| 184 | // Same config as before - nothing to clean. |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 185 | return |
| 186 | } |
| 187 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 188 | if config.Environment().IsEnvTrue("DISABLE_AUTO_INSTALLCLEAN") { |
| 189 | ctx.Println("DISABLE_AUTO_INSTALLCLEAN is set and true; skipping auto-clean. Your tree may be in an inconsistent state.") |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 190 | return |
| 191 | } |
| 192 | |
Nan Zhang | 17f2767 | 2018-12-12 16:01:49 -0800 | [diff] [blame] | 193 | ctx.BeginTrace(metrics.PrimaryNinja, "installclean") |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 194 | defer ctx.EndTrace() |
| 195 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 196 | previousProductAndVariant := strings.TrimPrefix(strings.TrimSuffix(previousConfig, suffix), prefix) |
| 197 | currentProductAndVariant := strings.TrimPrefix(strings.TrimSuffix(currentConfig, suffix), prefix) |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 198 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 199 | ctx.Printf("Build configuration changed: %q -> %q, forcing installclean\n", previousProductAndVariant, currentProductAndVariant) |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 200 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 201 | installClean(ctx, config) |
Dan Willemsen | f052f78 | 2017-05-18 15:29:04 -0700 | [diff] [blame] | 202 | |
| 203 | writeConfig() |
| 204 | } |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 205 | |
| 206 | // cleanOldFiles takes an input file (with all paths relative to basePath), and removes files from |
| 207 | // the filesystem if they were removed from the input file since the last execution. |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 208 | func cleanOldFiles(ctx Context, basePath, newFile string) { |
| 209 | newFile = filepath.Join(basePath, newFile) |
| 210 | oldFile := newFile + ".previous" |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 211 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 212 | if _, err := os.Stat(newFile); err != nil { |
| 213 | ctx.Fatalf("Expected %q to be readable", newFile) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 214 | } |
| 215 | |
| 216 | if _, err := os.Stat(oldFile); os.IsNotExist(err) { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 217 | if err := os.Rename(newFile, oldFile); err != nil { |
| 218 | ctx.Fatalf("Failed to rename file list (%q->%q): %v", newFile, oldFile, err) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 219 | } |
| 220 | return |
| 221 | } |
| 222 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 223 | var newData, oldData []byte |
| 224 | if data, err := ioutil.ReadFile(newFile); err == nil { |
| 225 | newData = data |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 226 | } else { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 227 | ctx.Fatalf("Failed to read list of installable files (%q): %v", newFile, err) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 228 | } |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 229 | if data, err := ioutil.ReadFile(oldFile); err == nil { |
| 230 | oldData = data |
| 231 | } else { |
| 232 | ctx.Fatalf("Failed to read list of installable files (%q): %v", oldFile, err) |
| 233 | } |
| 234 | |
| 235 | // Common case: nothing has changed |
| 236 | if bytes.Equal(newData, oldData) { |
| 237 | return |
| 238 | } |
| 239 | |
| 240 | var newPaths, oldPaths []string |
| 241 | newPaths = strings.Fields(string(newData)) |
| 242 | oldPaths = strings.Fields(string(oldData)) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 243 | |
| 244 | // These should be mostly sorted by make already, but better make sure Go concurs |
| 245 | sort.Strings(newPaths) |
| 246 | sort.Strings(oldPaths) |
| 247 | |
| 248 | for len(oldPaths) > 0 { |
| 249 | if len(newPaths) > 0 { |
| 250 | if oldPaths[0] == newPaths[0] { |
| 251 | // Same file; continue |
| 252 | newPaths = newPaths[1:] |
| 253 | oldPaths = oldPaths[1:] |
| 254 | continue |
| 255 | } else if oldPaths[0] > newPaths[0] { |
| 256 | // New file; ignore |
| 257 | newPaths = newPaths[1:] |
| 258 | continue |
| 259 | } |
| 260 | } |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 261 | |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 262 | // File only exists in the old list; remove if it exists |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 263 | oldPath := filepath.Join(basePath, oldPaths[0]) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 264 | oldPaths = oldPaths[1:] |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 265 | |
| 266 | if oldFile, err := os.Stat(oldPath); err == nil { |
| 267 | if oldFile.IsDir() { |
| 268 | if err := os.Remove(oldPath); err == nil { |
| 269 | ctx.Println("Removed directory that is no longer installed: ", oldPath) |
| 270 | cleanEmptyDirs(ctx, filepath.Dir(oldPath)) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 271 | } else { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 272 | ctx.Println("Failed to remove directory that is no longer installed (%q): %v", oldPath, err) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 273 | ctx.Println("It's recommended to run `m installclean`") |
| 274 | } |
| 275 | } else { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 276 | // Removing a file, not a directory. |
| 277 | if err := os.Remove(oldPath); err == nil { |
| 278 | ctx.Println("Removed file that is no longer installed: ", oldPath) |
| 279 | cleanEmptyDirs(ctx, filepath.Dir(oldPath)) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 280 | } else if !os.IsNotExist(err) { |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 281 | ctx.Fatalf("Failed to remove file that is no longer installed (%q): %v", oldPath, err) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 282 | } |
| 283 | } |
| 284 | } |
| 285 | } |
| 286 | |
| 287 | // Use the new list as the base for the next build |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 288 | os.Rename(newFile, oldFile) |
Dan Willemsen | 1e775d7 | 2020-01-03 13:40:45 -0800 | [diff] [blame] | 289 | } |
Dan Willemsen | 46459b0 | 2020-02-13 14:37:15 -0800 | [diff] [blame] | 290 | |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 291 | // cleanEmptyDirs will delete a directory if it contains no files. |
| 292 | // If a deletion occurs, then it also recurses upwards to try and delete empty parent directories. |
Dan Willemsen | 46459b0 | 2020-02-13 14:37:15 -0800 | [diff] [blame] | 293 | func cleanEmptyDirs(ctx Context, dir string) { |
| 294 | files, err := ioutil.ReadDir(dir) |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 295 | if err != nil { |
| 296 | ctx.Println("Could not read directory while trying to clean empty dirs: ", dir) |
Dan Willemsen | 46459b0 | 2020-02-13 14:37:15 -0800 | [diff] [blame] | 297 | return |
| 298 | } |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 299 | if len(files) > 0 { |
| 300 | // Directory is not empty. |
| 301 | return |
Dan Willemsen | 46459b0 | 2020-02-13 14:37:15 -0800 | [diff] [blame] | 302 | } |
Rupert Shuttleworth | 1f304e6 | 2020-11-24 14:13:41 +0000 | [diff] [blame] | 303 | |
| 304 | if err := os.Remove(dir); err == nil { |
| 305 | ctx.Println("Removed empty directory (may no longer be installed?): ", dir) |
| 306 | } else { |
| 307 | ctx.Fatalf("Failed to remove empty directory (which may no longer be installed?) %q: (%v)", dir, err) |
| 308 | } |
| 309 | |
| 310 | // Try and delete empty parent directories too. |
Dan Willemsen | 46459b0 | 2020-02-13 14:37:15 -0800 | [diff] [blame] | 311 | cleanEmptyDirs(ctx, filepath.Dir(dir)) |
| 312 | } |