Skip to content

Conversation

@luoliwoshang
Copy link
Member

@luoliwoshang luoliwoshang commented Mar 7, 2025

llcppg v0.2.0

conan install --requires sqlite3/3.49.1 --generator PkgConfigDeps --options=*:shared=True --build=missing --output-folder=./build && export PKG_CONFIG_PATH=$PWD/build:$PKG_CONFIG_PATH

goplus/llcppg#213

linux's go result is same with macos.

but the llcppsymg.symb.json have some different
+ is linux

 {
                "mangle":       "sqlite3_vmprintf",
-               "c++":  "sqlite3_vmprintf(const char *, int)",
+               "c++":  "sqlite3_vmprintf(const char *, va_list)",
                "go":   "Vmprintf"
        }, {
                "mangle":       "sqlite3_vsnprintf",
-               "c++":  "sqlite3_vsnprintf(int, char *, const char *, int)",
+               "c++":  "sqlite3_vsnprintf(int, char *, const char *, va_list)",
                "go":   "Vsnprintf"
        },

the hfile is same

SQLITE_API char *sqlite3_mprintf(const char*,...);
SQLITE_API char *sqlite3_vmprintf(const char*, va_list);
SQLITE_API char *sqlite3_snprintf(int,char*,const char*, ...);
SQLITE_API char *sqlite3_vsnprintf(int,char*,const char*, va_list);

#9 #7 every result is same in macos & linux

@luoliwoshang luoliwoshang marked this pull request as draft March 7, 2025 09:59
@luoliwoshang luoliwoshang force-pushed the sqlite branch 3 times, most recently from 7211720 to 5e9b802 Compare March 10, 2025 07:00
@luoliwoshang luoliwoshang force-pushed the sqlite branch 2 times, most recently from 8e9136e to f7e082e Compare March 13, 2025 09:42
@MeteorsLiu
Copy link
Contributor

This llpkg missed llpkg.cfg, which cannot pass Github Action verfication.
Here's the examplellpkg.cfg for this llpkg:

{
  "upstream": {
    "package": {
      "name": "sqlite3",
      "version": "3.36.0"
    }
  }
}

@luoliwoshang luoliwoshang force-pushed the sqlite branch 2 times, most recently from 70de76c to bcd1051 Compare March 18, 2025 03:43
@luoliwoshang luoliwoshang changed the title sqlite3/3.36.0 sqlite3/3.49.1 Mar 24, 2025
@luoliwoshang luoliwoshang marked this pull request as ready for review April 1, 2025 03:34
@aofei aofei merged commit 39cabc4 into goplus:main Apr 1, 2025
3 checks passed
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.

4 participants