Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-18632

Some header files are missing from debian packages

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.17.0
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      Kernel-related header files (uapi) are packaged in 'lustre-client-modules'
      Debian package, causing them to be unavailable when using dkms package instead.
          
      Move them in 'lustre-dev' along the other headers. This is on par with what is done for RPMs where those files are in 'lustre-devel' RPM package.

      lustre-dev:

      ./
      ./usr/
      ./usr/include/
      ./usr/include/linux/
      ./usr/include/linux/lnet/
      ./usr/include/linux/lnet/libcfs_debug.h
      ./usr/include/linux/lnet/libcfs_ioctl.h
      ./usr/include/linux/lnet/lnet-dlc.h
      ./usr/include/linux/lnet/lnet-types.h
      ./usr/include/linux/lnet/lnetctl.h
      ./usr/include/linux/lnet/lnetst.h
      ./usr/include/linux/lnet/nidstr.h
      ./usr/include/linux/lnet/socklnd.h
      ./usr/include/linux/lustre/
      ./usr/include/linux/lustre/lgss.h
      ./usr/include/linux/lustre/lustre_cfg.h
      ./usr/include/linux/lustre/lustre_disk.h
      ./usr/include/linux/lustre/lustre_fid.h
      ./usr/include/linux/lustre/lustre_fiemap.h
      ./usr/include/linux/lustre/lustre_idl.h
      ./usr/include/linux/lustre/lustre_ioctl.h
      ./usr/include/linux/lustre/lustre_kernelcomm.h
      ./usr/include/linux/lustre/lustre_ostid.h
      ./usr/include/linux/lustre/lustre_param.h
      ./usr/include/linux/lustre/lustre_user.h
      ./usr/include/linux/lustre/lustre_ver.h
      ./usr/include/lustre/
      ./usr/include/lustre/liblustreapi.h
      ./usr/include/lustre/ll_fiemap.h
      ./usr/include/lustre/lustre_barrier_user.h
      ./usr/include/lustre/lustre_lfsck_user.h
      ./usr/include/lustre/lustre_user.h
      ./usr/include/lustre/lustreapi.h
      ./usr/lib/
      ./usr/lib/liblnetconfig.a
      ./usr/lib/liblustreapi.a
      ./usr/lib/pkgconfig/
      ./usr/lib/pkgconfig/lustre.pc
      ./usr/share/
      ./usr/share/doc/
      ./usr/share/doc/lustre-dev/
      ./usr/share/doc/lustre-dev/changelog.Debian.gz
      ./usr/share/doc/lustre-dev/changelog.gz
      ./usr/share/doc/lustre-dev/contrib/
      ./usr/share/doc/lustre-dev/copyright
      ./usr/lib/liblnetconfig.so
      ./usr/lib/liblustreapi.so

      lustre-devel (RPM)

      $ rpm -qlp lustre-client-devel-2.16.1-1.el9.x86_64.rpm 
      /usr/include/linux/lnet
      /usr/include/linux/lnet/libcfs_debug.h
      /usr/include/linux/lnet/libcfs_ioctl.h
      /usr/include/linux/lnet/lnet-crypto.h
      /usr/include/linux/lnet/lnet-dlc.h
      /usr/include/linux/lnet/lnet-idl.h
      /usr/include/linux/lnet/lnet-nl.h
      /usr/include/linux/lnet/lnet-types.h
      /usr/include/linux/lnet/lnetctl.h
      /usr/include/linux/lnet/lnetst.h
      /usr/include/linux/lnet/nidstr.h
      /usr/include/linux/lnet/socklnd.h
      /usr/include/linux/lustre
      /usr/include/linux/lustre/lgss.h
      /usr/include/linux/lustre/lustre_cfg.h
      /usr/include/linux/lustre/lustre_fid.h
      /usr/include/linux/lustre/lustre_fiemap.h
      /usr/include/linux/lustre/lustre_idl.h
      /usr/include/linux/lustre/lustre_ioctl.h
      /usr/include/linux/lustre/lustre_kernelcomm.h
      /usr/include/linux/lustre/lustre_ostid.h
      /usr/include/linux/lustre/lustre_param.h
      /usr/include/linux/lustre/lustre_user.h
      /usr/include/linux/lustre/lustre_ver.h
      /usr/include/lustre
      /usr/include/lustre/liblustreapi.h
      /usr/include/lustre/ll_fiemap.h
      /usr/include/lustre/lustre_barrier_user.h
      /usr/include/lustre/lustre_lfsck_user.h
      /usr/include/lustre/lustre_user.h
      /usr/include/lustre/lustreapi.h
      /usr/lib64/liblnetconfig.so
      /usr/lib64/liblustreapi.a
      /usr/lib64/liblustreapi.so
      /usr/lib64/pkgconfig/lustre.pc

      Attachments

        Activity

          [LU-18632] Some header files are missing from debian packages
          pjones Peter Jones added a comment -

          Merged for 2.17

          pjones Peter Jones added a comment - Merged for 2.17

          "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57749/
          Subject: LU-18632 debian: add uapi headers to dkms module package
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 455bde17d1572ae25593182dd80d31046888e2ed

          gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57749/ Subject: LU-18632 debian: add uapi headers to dkms module package Project: fs/lustre-release Branch: master Current Patch Set: Commit: 455bde17d1572ae25593182dd80d31046888e2ed

          The reason for this is due to the native Linux client. If we build with --disable-modules then the uapi headers shouldn't be included in the dpkg. I guess with modules enabled we want to over ride the Linux client UAPI header versions.

          simmonsja James A Simmons added a comment - The reason for this is due to the native Linux client. If we build with --disable-modules then the uapi headers shouldn't be included in the dpkg. I guess with modules enabled we want to over ride the Linux client UAPI header versions.

          "Aurelien Degremont <adegremont@nvidia.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57749
          Subject: LU-18632 debian: put header files in 'lustre-dev'
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: b4afbf23bdd8b964a0bff67d769f79d8de67f0a5

          gerrit Gerrit Updater added a comment - "Aurelien Degremont <adegremont@nvidia.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57749 Subject: LU-18632 debian: put header files in 'lustre-dev' Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: b4afbf23bdd8b964a0bff67d769f79d8de67f0a5

          People

            adegremont_nvda Aurelien Degremont
            adegremont_nvda Aurelien Degremont
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: