Skip to content

Commit

Permalink
[bugfix] Include platform to ldlogger.so path
Browse files Browse the repository at this point in the history
A previous improvement in commit db924b0 requires that ldlogger.so
file is available in a directory named the current platform. When
building a Python package from CodeChecker, this ldlogger.so is not
placed in such a directory.
This patch fixes this issue.
  • Loading branch information
bruntib committed Aug 10, 2023
1 parent 38f9315 commit 7b92aed
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion setup.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#!/usr/bin/env python3

import os
import platform
import setuptools
import subprocess
import sys
Expand Down Expand Up @@ -102,7 +103,7 @@ def init_packages():

class BuildExt(build_ext):
def get_ext_filename(self, ext_name):
return f"{os.path.join(*ext_name.split('.'))}.so"
return f"{platform.uname().machine}/{ext_name}.so"

def build_extension(self, ext):
if sys.platform == "linux":
Expand Down

0 comments on commit 7b92aed

Please sign in to comment.