roxygenize fails with missing NAMESPACE when using setMethod

[hdarjus]

I would like to re-open #1144 here, because #1538 seems to be inadequate for the case in the title.

I have created an MWE under https://github.com/hdarjus/badInstall. roxygen2::roxygenize() fails for the package as is (Case 1). However, after renaming NAMESPACE2 to NAMESPACE (Case 2), roxygen2::roxygenize() works even though the imports are technically the same: there are no imports in either cases.

I believe the issue is !file.exists(NAMESPACE) in

roxygen2/R/namespace.R

Line 72 in 9652d15

if (!made_by_roxygen(NAMESPACE) || !file.exists(NAMESPACE)) {

I think something similar to the following modified function could work:

update_namespace_imports <- function(base_path) {
  NAMESPACE <- file.path(base_path, "NAMESPACE")
  if (!made_by_roxygen(NAMESPACE)) {
    return(invisible())
  }

  local_namespace_exports <- if(!file.exists(NAMESPACE)) {
    character(0)
  } else {
     namespace_exports(NAMESPACE)
  }

  lines <- c(namespace_imports(base_path), local_namespace_exports)
  results <- c(made_by("#"), sort_c(unique(trimws(lines))))
  write_if_different(NAMESPACE, results, check = TRUE)  # maybe check again whether NAMESPACE exists

  invisible()
}

I did not have the time to test it, unfortunately.