Skip to content

Add additional libssc.so call to get alma linux build working#2255

Closed
brtietz wants to merge 1 commit into
patchfrom
btm_alma_linux_updates
Closed

Add additional libssc.so call to get alma linux build working#2255
brtietz wants to merge 1 commit into
patchfrom
btm_alma_linux_updates

Conversation

@brtietz

@brtietz brtietz commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Add libssc.so to release build path for alma linux build.

@brtietz
brtietz requested a review from sjanzou July 21, 2026 21:05

@sjanzou sjanzou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In all of the releases to date, ssc.so and not libssc.so is used, e.g. in the 2026.7.3 release:

Image

Can you please remove the libssc.so find library call and build again with SAMAPI_EXPORT=0?

Thanks!

Comment thread CMakeLists.txt
PATHS $ENV{LK_LIB} $ENV{LKDIR}/build $ENV{LKDIR}/build/Release)
find_library( SSC_LIB
NAMES ssc.dylib ssc.lib ssc.so
NAMES ssc.dylib ssc.lib ssc.so libssc.so

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not sure why you need libssc.so. Historically, we have used ssc.so for all releases and libssc.so is only generated or looked for when SAMAPI_EXPORT is defined. Typical uses are pysam and HPC or pvwatts standalone builds of ssc.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It looks like the key command is --target ssc as a part of the make command. I'll close this PR and think about how to make that more obvious in the instructions.

@brtietz brtietz closed this Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants