Unregister package before installing instead of unloading (#2437)

Closes #2349
r-lib · Jul 20, 2022 · 7f4aa1f · 7f4aa1f
1 parent 4e3b7c3
commit 7f4aa1f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/NEWS.md b/NEWS.md
@@ -1,5 +1,10 @@
 # devtools (development version)
 
+* `install(reload = TRUE)` now calls `pkgload::unregister()` instead
+  of `pkload::unload()` (#2349). This allows the package to keep
+  functioning if it is still in use in the R session (e.g. through
+  event handlers).
+
 * `test()` no longer calls `load_all()` twice. `test_active_file()`
   now calls `load_all()` via testthat.
 

diff --git a/R/install.R b/R/install.R
@@ -96,7 +96,7 @@ install <-
     was_attached <- is_attached(pkg)
 
     if (reload && was_loaded) {
-      pkgload::unload(pkg$package)
+      pkgload::unregister(pkg$package)
     }
 
     pkgbuild::with_build_tools(required = FALSE,