daef / poly-bool-comparison

10 stars 4 forks source link

feat(all): adjusted package configurations, and added support for management via lerna lite #9

Closed z3dev closed 1 year ago

z3dev commented 1 year ago

Given that there are many sub-packages, I suggest moving to lerna lite for managing each of the packages, which should make it easy to build, and test later.

As a bonus, there's a new package.json for dissection which has several scripts to setup / cleanup linkages to the other sub-packages, and start 'dev' webserver.

daef commented 1 year ago

I've never heard about lerna. I should probably read up on that a bit :)

daef commented 1 year ago

I like what I see about lerna, trying to ./run I first had to change --export-png to --export-filename for current inkscape, but it still fails pretty hard.

z3dev commented 1 year ago

I like what I see about lerna, trying to ./run I first had to change --export-png to --export-filename for current inkscape, but it still fails pretty hard.

Yeah. I have that change as well. The current Inkscape is very different on the command line.

These changes are focused on the initial setup of each sub-project. Although, I couldn't determine how to set up some of the packages, like the RUST library.

We can add more scripts to do almost anything across the packages. I will add a few in another pull request. I hope that @platypul also likes this direction.

z3dev commented 1 year ago

@daef What do you think? Anything that should be changed?

daef commented 1 year ago

is it already running though? o0

z3dev commented 1 year ago

Sure. Here's the output of running 'npm run setup'

npm run setup

> poly-bool-comparison@1.0.0 setup
> lerna run setup

lerna-lite notice cli v1.14.1
lerna-lite info Executing command in 4 packages: "npm run setup"
lerna-lite info run Ran npm script 'setup' in 'pathkit' in 0.6s:

> pathkit@1.0.0 setup
> npm install

up to date, audited 2 packages in 237ms

found 0 vulnerabilities
lerna-lite info run Ran npm script 'setup' in 'voidqk' in 0.6s:

> voidqk@1.0.0 setup
> npm install

up to date, audited 2 packages in 237ms

found 0 vulnerabilities
lerna-lite info run Ran npm script 'setup' in 'dissection' in 0.6s:

> dissection@1.0.0 setup
> npm install && npm run create-links

up to date, audited 1 package in 37ms

found 0 vulnerabilities

> dissection@1.0.0 create-links
> ln -sf ../magick && ln -sf ../martinez_cgal && ln -sf ../martinez_cpp && ln -sf ../milevski && ln -sf ../pathkit && ln -sf ../rust-geo-booleanop && ln -sf ../unisamples && ln -sf ../voidqk

lerna-lite info run Ran npm script 'setup' in 'milevski' in 0.9s:

> milevski@1.0.0 setup
> npm install

up to date, audited 5 packages in 556ms

found 0 vulnerabilities
lerna-lite success run Ran npm script 'setup' in 4 packages in 1.0s:
lerna-lite success - dissection
lerna-lite success - milevski
lerna-lite success - pathkit
lerna-lite success - voidqk
npm notice 
npm notice New minor version of npm available! 9.5.0 -> 9.8.1
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.8.1
npm notice Run npm install -g npm@9.8.1 to update!
npm notice 
daef commented 1 year ago

oh, then i guess i'll have to try this :)

z3dev commented 1 year ago

@daef please try.

I updated the README to include the steps in the installation process. It's actually very simple now.

daef commented 1 year ago

I'm on a trip. I'll be back in a week! Looking forward to :)

daef commented 1 year ago

sorry, the whole family was sick - cloning your branch right now

daef commented 1 year ago

seems to run through just nice after a few small changes. I'll merge it and push my changes on top :)