ClobberError with hdf4

[dhirschfeld]

Although it seems harmless enough I thought I'd report it

(ClobberError: This transaction has incompatible packages due to a shared path.
  packages: conda-forge::hdf5-1.10.1-vc14_2, conda-forge::hdf4-4.2.13-vc14_0
  path: 'library/copying'

ClobberError: This transaction has incompatible packages due to a shared path.
  packages: conda-forge::hdf5-1.10.1-vc14_2, conda-forge::hdf4-4.2.13-vc14_0
  path: 'library/release.txt'

λ conda create -v -n deleteme999 python=3.6* conda-forge::hdf5 conda-forge::hdf4
Solving environment: ...working... done

## Package Plan ##

  environment location: C:\Miniconda3\envs\deleteme999

  added / updated specs:
    - conda-forge::hdf4
    - conda-forge::hdf5
    - python=3.6


The following NEW packages will be INSTALLED:

    certifi:        2018.4.16-py36_0
    hdf4:           4.2.13-vc14_0      conda-forge [vc14]
    hdf5:           1.10.1-vc14_2      conda-forge [vc14]
    jpeg:           9b-vc14_2          conda-forge [vc14]
    pip:            10.0.1-py36_0
    python:         3.6.5-h0c2934d_0
    setuptools:     39.1.0-py36_0
    vc:             14-h0510ff6_3
    vs2015_runtime: 14.0.25420-0
    wheel:          0.31.1-py36_0
    wincertstore:   0.2-py36h7fe50ca_0
    zlib:           1.2.11-vc14_0      conda-forge [vc14]

Proceed ([y]/n)? y

Preparing transaction: ...working... done
Verifying transaction: ...working... (ClobberError: This transaction has incompatible packages due to a shared path.
  packages: conda-forge::hdf5-1.10.1-vc14_2, conda-forge::hdf4-4.2.13-vc14_0
  path: 'library/copying'

, ClobberError: This transaction has incompatible packages due to a shared path.
  packages: conda-forge::hdf5-1.10.1-vc14_2, conda-forge::hdf4-4.2.13-vc14_0
  path: 'library/release.txt'

)

done
Executing transaction: ...working... ===> LINKING PACKAGE: defaults::vs2015_runtime-14.0.25420-0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\vs2015_runtime-14.0.25420-0


===> LINKING PACKAGE: defaults::vc-14-h0510ff6_3 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\vc-14-h0510ff6_3


===> LINKING PACKAGE: conda-forge::zlib-1.2.11-vc14_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\zlib-1.2.11-vc14_0


===> LINKING PACKAGE: conda-forge::hdf5-1.10.1-vc14_2 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\hdf5-1.10.1-vc14_2


===> LINKING PACKAGE: conda-forge::jpeg-9b-vc14_2 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\jpeg-9b-vc14_2


===> LINKING PACKAGE: defaults::python-3.6.5-h0c2934d_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\python-3.6.5-h0c2934d_0


===> LINKING PACKAGE: defaults::certifi-2018.4.16-py36_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\certifi-2018.4.16-py36_0


===> LINKING PACKAGE: conda-forge::hdf4-4.2.13-vc14_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\hdf4-4.2.13-vc14_0


file exists, but clobbering: 'C:\\Miniconda3\\envs\\deleteme999\\Library\\COPYING'

file exists, but clobbering: 'C:\\Miniconda3\\envs\\deleteme999\\Library\\RELEASE.txt'

===> LINKING PACKAGE: defaults::wincertstore-0.2-py36h7fe50ca_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\wincertstore-0.2-py36h7fe50ca_0


===> LINKING PACKAGE: defaults::setuptools-39.1.0-py36_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\setuptools-39.1.0-py36_0


===> LINKING PACKAGE: defaults::wheel-0.31.1-py36_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\wheel-0.31.1-py36_0


===> LINKING PACKAGE: defaults::pip-10.0.1-py36_0 <===
  prefix=C:\Miniconda3\envs\deleteme999
  source=C:\Miniconda3\pkgs\pip-10.0.1-py36_0


done

[ocefpaf]

Thanks! That is not too bad b/c the license that is getting clobbered does exist somewhere else in the coda package metadata. Still, we should fix this by moving files to another directory.

[hmaarrfk]

in my mind, that file is most easily removed.

We should already be including it in the info directory, so it seems a ltitle redundant.

Would a PR removing the file be acceptable?

[dhirschfeld]

Sounds reasonable to me.

[hmaarrfk]

See conda-forge/hdf4-feedstock#30
and #143

[hmaarrfk]

Apparently, we can set HDF5_INSTALL_DATA_DIR to control the location of these files:

-----------------------------------------------------------------------------
# Add Document File(s) to CMake Install
#-----------------------------------------------------------------------------
if (NOT HDF5_EXTERNALLY_CONFIGURED)
  install (
      FILES ${HDF5_SOURCE_DIR}/COPYING
      DESTINATION ${HDF5_INSTALL_DATA_DIR}
      COMPONENT hdfdocuments
  )
  if (EXISTS "${HDF5_SOURCE_DIR}/release_docs" AND IS_DIRECTORY "${HDF5_SOURCE_DIR}/release_docs")
    set (release_files
        ${HDF5_SOURCE_DIR}/release_docs/USING_HDF5_CMake.txt
        ${HDF5_SOURCE_DIR}/release_docs/COPYING
        ${HDF5_SOURCE_DIR}/release_docs/RELEASE.txt
    )
    if (WIN32 OR MINGW)
      set (release_files
          ${release_files}
          ${HDF5_SOURCE_DIR}/release_docs/USING_HDF5_VS.txt
      )
    endif ()
    if (HDF5_PACK_INSTALL_DOCS)
      set (release_files
          ${release_files}
          ${HDF5_SOURCE_DIR}/release_docs/INSTALL_Warnings.txt
          ${HDF5_SOURCE_DIR}/release_docs/INSTALL_CMake.txt
          ${HDF5_SOURCE_DIR}/release_docs/HISTORY-1_8.txt
          ${HDF5_SOURCE_DIR}/release_docs/INSTALL
      )
      if (WIN32 OR MINGW)
        set (release_files
            ${release_files}
            ${HDF5_SOURCE_DIR}/release_docs/INSTALL_Windows.txt
        )
      endif ()
      if (CYGWIN)
        set (release_files
            ${release_files}
            ${HDF5_SOURCE_DIR}/release_docs/INSTALL_Cygwin.txt
        )
      endif ()
      if (HDF5_ENABLE_PARALLEL)
        set (release_files
            ${release_files}
            ${HDF5_SOURCE_DIR}/release_docs/INSTALL_parallel
        )
      endif ()
    endif ()
    install (
        FILES ${release_files}
        DESTINATION ${HDF5_INSTALL_DATA_DIR}
        COMPONENT hdfdocuments
    )
  endif ()
endif ()

Do you have a good location for them in mind?