What snapshot is this? Could be this a dupe of https://bugs.swift.org/browse/SR-3816 ?

PS: top level modulemap in https://github.com/amraboelela/CLevelDB makes it a system package

Comment by Amr Aboelela (JIRA)

I don't use a snapshot, I make a daily build from source using "./swift/utils/build-toolchain local.swift" script, it takes 13 hours 😃

Is it wrong that it is a system package?

I didn't understand the relation with SR-3816 ? am I missing something in the modulemap ?

can you tell me the revision of swiftpm that you built?
no its not wrong, just that CLevelDB needs to be built manually and installed before it can be linked. (importing works fine because the headers can be seen)

Comment by Amr Aboelela (JIRA)

It is already built and installed:

/usr/lib$ ls -l
...
lrwxrwxrwx 1 root root 19 Feb 6 10:48 libCLevelDB.so -> libCLevelDB.so.1.19
-rwxr-xr-x 1 root root 455K Feb 6 10:48 libCLevelDB.so.1.19
lrwxrwxrwx 1 root root 19 Feb 6 10:48 libCLevelDB.so.1 -> libCLevelDB.so.1.19

Also I am using a toolchain which is build just yesterday, which means it should include this PR #927 already, but I am still getting the same error.

Comment by Amr Aboelela (JIRA)

This is the swiftpm I have:

swiftpm$ git log
commit fad1b0c
Author: Ankit Aggarwal <ankit_aggarwal@apple.com>
Date: Sun Feb 5 12:17:57 2017 +0530

[PackageModel] Minor cleanup in Product

commit a0e044a
Merge: b25030a cb70454
Author: swift-ci <swift-ci@users.noreply.github.com>
Date: Sat Feb 4 09:08:03 2017 -0800

Merge pull request #924 from Steveybrown/process-thread-safe

commit cb70454
Author: Stephen Brown <steveybrown@gmail.com>
Date: Mon Jan 30 20:57:31 2017 +0000

[Utility] Added thread safety to Process.

• https://bugs.swift.org/browse/SR-3625

Comment by Amr Aboelela (JIRA)

Oh, so it was you who made the last commit 😉

I can't reproduce this on current master but I can investigate this if you provide revision of swiftpm that you built. Otherwise, can you try using a trunk or swift-3.1 toolchain from swift.org and see if it still reproduces for you?

Thanks, I'll check. was this on macOS or linux?

Comment by Amr Aboelela (JIRA)

I am already using the latest thing from master. It is in Linux Ubuntu 16.04

ah right

ok I see, this works fine on macOS (on which I initially tested) but not on linux because we're passing LinuxMain.swift in link line which makes it want to see all modules again.
We need to compile LinuxMain as a swift target and then include its object while linking.

This should fix it: #936

Thanks for reporting!

Comment by Amr Aboelela (JIRA)

Ok thanks. But why was it working fine till Dec 2016 ?

The Build module was rewritten to be more testable and maintainable, this is a usecase we don't have any test for but now I'll add one so it doesn't regress 🙂

Comment by Amr Aboelela (JIRA)

I am finally able to verify the fix, thank you.