Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

src: move some code under comp/ #720

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

src: move some code under comp/ #720

wants to merge 2 commits into from

Commits on Jul 20, 2024

  1. src: move Parsec/ under comp/ 

    The top-level structure under `src/` is awkward and it's unclear why Parsec is
here. Why not `vendor/`? Ultimately though, I think the best place is actually
inside the source code, because it's not like anything else uses it and it's
`bsc`-owned code already in practice anyway.

Signed-off-by: Austin Seipp <[email protected]>
    @thoughtpolice
    thoughtpolice committed Jul 20, 2024
    Configuration menu
    Copy the full SHA
    c0b6fcf View commit details
    Browse the repository at this point in the history

  2. src: move vendor/htcl/ under comp/ 

    Similarly to Parsec, this code is practically only used in one spot, so it
might as well be maintained there. I don't even know what codebase HTcl is
a 'vendoring' *of* and it's dated to over 15 years ago at minimum, so it's
probably lost to history now... Just put it all together for clarity and ease
of development.

Signed-off-by: Austin Seipp <[email protected]>
    @thoughtpolice
    thoughtpolice committed Jul 20, 2024
    Configuration menu
    Copy the full SHA
    c778cc8 View commit details
    Browse the repository at this point in the history