Skip to content

Conversation

@Bhavdeep0108
Copy link
Collaborator

No description provided.

- Added pjsip-apps/src/pjsua to include_dirs for pjsua-lib/pjsua.h
- Fixed library path using node path.resolve() for cross-platform compatibility
- Build now works locally and should work in CI environment
- Added library copying step in workflow to ensure PJSIP libs are available for Node addon build
- Enhanced binding.gyp with file existence check for better CI error handling
- Fixed include paths and library paths for cross-platform compatibility
- Build now works both locally and in GitHub Actions CI environment
- Changed MSBuild platform from win32 to x64 in GitHub Actions workflow
- Updated binding.gyp to dynamically detect architecture and use correct library name
- Now builds x86_64-x64 libraries instead of i386-Win32 libraries
- Fixes 'lib not found' error in CI environment
- Build now works for both 32-bit and 64-bit architectures
- Added clean step to remove previous x86 build artifacts
- Build only core PJSIP projects individually to avoid mixed architecture
- Force all projects to build for x64 architecture
- Skip problematic third-party projects that cause x86/x64 conflicts
- This resolves the 'module machine type x86 conflicts with target machine type x64' errors
@InnovateAsterisk InnovateAsterisk merged commit 3b4767a into main Sep 25, 2025
1 check failed
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.

3 participants