monkey2 icon indicating copy to clipboard operation
monkey2 copied to clipboard

Module Manager error?

Open DruggedBunny opened this issue 6 years ago • 7 comments

I just clicked to install png2polygon from Ted2Go's Module Manager, and it appears to be rebuilding all modules, including SDL2, assimp, stb-vorbis, std, etc!

Guessing that's not meant to happen, certainly never done that before!

image

DruggedBunny avatar Jan 17 '19 18:01 DruggedBunny

I have also noticed this and thought it may have been a bug . . . I wonder if this is an IDE or tool related thing.

XANOZOID avatar Jan 17 '19 23:01 XANOZOID

IDE update modules, don't rebuild from scratch. I think proper way is to update only modules from depends section.

engor avatar Jan 18 '19 04:01 engor

Yes, after a module manager update, monkey2 just does an 'update modules', and the first time this is done all modules end up getting built.

I think proper way is to update only modules from depends section.

Actually, I think just updating the installed/updated modules might be enough as this should automatically update dependancies?

On Fri, Jan 18, 2019 at 5:11 PM Evgeniy Goroshkin [email protected] wrote:

IDE update modules, don't rebuild from scratch. I think proper way is to update only modules from depends section.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/blitz-research/monkey2/issues/458#issuecomment-455421676, or mute the thread https://github.com/notifications/unsubscribe-auth/ADU3Qno3n6PXBbNPBEZq2uxpbolo92iuks5vEUlugaJpZM4aGeWs .

blitz-research avatar Jan 18 '19 23:01 blitz-research

Is that new? I know in the past (quite some time ago, admittedly) it didn't seem to do this. I actually thought it might be weirdly related to the rebuildall*.bat changes recently, eg. running updateall in error.

I didn't want to cancel the build, not knowing what state things would have been left in, and basically had to find something else to do for half an hour or so! (#firstworldproblems)

It's hard to see the need to rebuild std-vorbis or assimp when adding png2polygon, for instance...

DruggedBunny avatar Jan 19 '19 03:01 DruggedBunny

Actually, I think just updating the installed/updated modules might be enough as this should automatically update dependancies?

OK.

I didn't want to cancel the build, not knowing what state things would have been left in, and basically had to find something else to do for half an hour or so! (#firstworldproblems)

Yes, if you use precompiled version (and many of users use that) then first update is the same as total rebuild coz there is no built files yet.

engor avatar Jan 19 '19 04:01 engor

I just pushed fix into ted2go's dev-branch, since now it will update only performed modules.

engor avatar Jan 19 '19 06:01 engor

Can be closed?

seyhajin avatar Jun 04 '19 15:06 seyhajin