Skip to content

Commit 1dcfd45

Browse files
authored
Merge pull request #57 from JohanMabille/upgrade_nlohmann
Upgraded to nlohmann_json 3.12 and compatible xwidgets and xcanvas
2 parents 76593b4 + 096809f commit 1dcfd45

File tree

4 files changed

+8
-8
lines changed

4 files changed

+8
-8
lines changed

.github/workflows/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
- uses: actions/checkout@v4
3030

3131
- name: Install micromamba
32-
uses: mamba-org/setup-micromamba@v1
32+
uses: mamba-org/setup-micromamba@v2
3333
with:
3434
environment-file: environment-dev.yml
3535

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,8 @@ endif()
9191

9292

9393
set(xeus_REQUIRED_VERSION 5.0.0)
94-
set(xwidgets_REQUIRED_VERSION 0.29.0)
95-
set(xcanvas_REQUIRED_VERSION 0.5.0)
94+
set(xwidgets_REQUIRED_VERSION 0.29.2)
95+
set(xcanvas_REQUIRED_VERSION 0.6.1)
9696
set(xproperty_REQUIRED_VERSION 0.12.0)
9797
set(Lua_REQUIRED_VERSION 5.3.4)
9898

environment-dev.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@ dependencies:
1212
- cppzmq
1313
- lua>=5.2.0
1414
- openlibm # [linux]
15-
- xwidgets >=0.29
16-
- xcanvas>=0.4.2
15+
- xwidgets >=0.29.2
16+
- xcanvas>=0.6.1
1717
# to try it out
1818
- jupyterlab
1919
# Test dependencies

include/xeus-lua/xinterpreter.hpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@ extern "C" {
2727
# include "lauxlib.h"
2828
# include "lualib.h"
2929
#else
30-
#include "luajit-2.0/lua.h"
31-
#include "luajit-2.0/lauxlib.h"
32-
#include "luajit-2.0/lualib.h"
30+
#include "luajit-2.1/lua.h"
31+
#include "luajit-2.1/lauxlib.h"
32+
#include "luajit-2.1/lualib.h"
3333
#endif
3434
}
3535
namespace nl = nlohmann;

0 commit comments

Comments
 (0)