cromwell icon indicating copy to clipboard operation
cromwell copied to clipboard

Repo does not include the cromwell for xblastos

Open JayFoxRox opened this issue 8 years ago • 3 comments

It was brought to my attention that the 2015 xblastos modchip also runs cromwell: https://bitbucket.org/psyko_chewbacca/lpcmod_os . That cromwell repo seems to have support for modern toolchains (Don't be fooled by the README, check the makefile instead).

I'm not aware which changes were made.

We should consider moving the xblastos branches here too or at least link to it as it seems to be the latest / most maintained version of cromwell.

(Related to #1 )

JayFoxRox avatar May 13 '17 07:05 JayFoxRox

This repo - to my knowledge - is just an archive. We should keep the archive, but also somehow add the work by the xblast dev (which is build on top of what this repo offers).

As to why bitbucket: It's their choice.

JayFoxRox avatar Jul 28 '17 13:07 JayFoxRox

I've checked lpcmod_os repo, they have a really big bunch of new commits in master, excluding their separate branches with additional features. This would be an overkill to review everything.

I see only these options here:

  1. Take their code as is and then do work on top of it
  2. Check out their firmware features and decide what to import from them
  3. Don't touch their code at all

We should keep the archive

Just create a branch archive with HEAD at 5b06e83.

binarymaster avatar Jun 01 '19 23:06 binarymaster

"xblast os" seems to be purpose built for Aladdin modchips, so I don't think everything done on that fork will be relevant to this project, though it does look like they have some useful features. I think we should at some point review what features/bug fixes they have implemented and back port them as necessary.

mborgerson avatar Oct 10 '19 00:10 mborgerson