Skip to content

New package: slurm-drmaa#25424

Merged
bernhardkaindl merged 4 commits intospack:developfrom
spenpw:slurm-drmaa
Oct 1, 2021
Merged

New package: slurm-drmaa#25424
bernhardkaindl merged 4 commits intospack:developfrom
spenpw:slurm-drmaa

Conversation

@spenpw
Copy link
Copy Markdown
Contributor

@spenpw spenpw commented Aug 14, 2021

This adds a package script for slurm-drmaa.

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Aug 14, 2021

Hi @pwablito! I noticed that the following package(s) don't yet have maintainers:

  • slurm-drmaa

Are you interested in adopting any of these package(s)? If so, simply add the following to the package class:

    maintainers = ['pwablito']

If not, could you contact the developers of this package and see if they are interested? You can quickly see who has worked on a package with spack blame:

$ spack blame slurm-drmaa

Thank you for your help! Please don't add maintainers without their consent.

You don't have to be a Spack expert or package developer in order to be a "maintainer," it just gives us a list of users willing to review PRs or debug issues relating to this package. A package can have multiple maintainers; just add a list of GitHub handles of anyone who wants to volunteer.

@spenpw spenpw changed the title Slurm DRMAA package New package: slurm-drmaa Aug 14, 2021
@bernhardkaindl
Copy link
Copy Markdown
Contributor

@pwablito The slurm package builds, but the slurm-drmaa doesn't when compiled in a clean Ubuntu 18.04 container:

     107    checking for usable SLURM libraries/headers... *** The SLURM test program failed to link 
            or run. See the file config.log
     108    *** for the exact error that occured.
     109    no
  >> 110    configure: error:
     111    SLURM libraries/headers not found;
     112    add --with-slurm-inc and --with-slurm-lib with appropriate locations.

config.log

configure:13714: checking for usable SLURM libraries/headers
configure:13735: /home/kaindlb/spack/spack/lib/spack/env/gcc/gcc -o conftest -pedantic -std=c99 -g -O2
 -pthread -D_REENTRANT -D_THREAD_SAFE -DNDEBUG  -D_GNU_SOURCE -I/home/kaindlb/spack/spack/opt/spack/li
nux-ubuntu18.04-zen/gcc-8.4.0/slurm-21-08-1-1-6jbxvnvk5douefrouygeavqcetcvpw5z/include  -L/home/kaindl
b/spack/spack/opt/spack/linux-ubuntu18.04-zen/gcc-8.4.0/slurm-21-08-1-1-6jbxvnvk5douefrouygeavqcetcvpw
5z/lib conftest.c -pthread  -lslurmdb -lslurm  >&5
/usr/bin/ld: cannot find -lslurmdb

found in opt that libslurm is built, libslurmdb not.

@spackbot-app spackbot-app bot added the patch label Sep 30, 2021
Copy link
Copy Markdown
Contributor

@bernhardkaindl bernhardkaindl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed the build, notified upstream to merge PR natefoo/slurm-drmaa#62

@bernhardkaindl bernhardkaindl enabled auto-merge (squash) September 30, 2021 20:51
@bernhardkaindl bernhardkaindl merged commit 046ed47 into spack:develop Oct 1, 2021
haralmha pushed a commit to haralmha/spack that referenced this pull request Oct 1, 2021
Co-authored-by: Bernhard Kaindl <bernhardkaindl7@gmail.com>
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Oct 1, 2021
Co-authored-by: Bernhard Kaindl <bernhardkaindl7@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants