[WIP] Remote Image

[czystyl]

• Add httpkit
• Get image from url
• Show image
• Clean up the mess :D

[rauanmayemir]

This is a wrong place to add dependencies.

1. It's a monorepo config, so anything that depends on brisk-macos will fail to find those dependencies
2. It's devDependencies - meaning they are either for tooling like merlin, or some local deps you'd use for tests, they will not be available in runtime

We'll have a detailed design doc and contributing guide, but for now:

1. Add those deps to the brisk-macos
2. Also add the resolutions to the relevant example apps (we have just components for now)

[rauanmayemir]

Whatever you add to the resolutions, try to keep order alphabetical (scopes go first).

[wokalski]

I'm opposed to this rule. Any rule you cannot automate is just mental overhead!

EDIT: And it's all about how you can justify that mental overhead.

[rauanmayemir]

I'd prefer to sort them by significance but that's what would be a mental overhead, so alphabetical seems like a good compromise. (most package managers do it automatically, maybe one day we'll get that for esy)

It's not like we'll add dependencies on every single PR.

[wokalski]

I'm still opposed to it - I'll never remember/care about such artificial rule.

EDIT: I might even switch order on purpose 😂
EDIT2: re: "by significance". I could come up with an equally artificial rule - add the new one always on the bottom so that we know the order of new dependencies! And it would give as much value (i.e. none)

[wokalski]

Aren't there enough artificial rules in this world 😄 ?

[wokalski]

[rauanmayemir]

Ok, let's postpone the manual enforcing. We'll come up with a way to automate it.

[czystyl]

Don't fight guys :D Ok, what's about to keep the opam packages at the top and the rest of packages alphabetical?

[rauanmayemir]

That’s what I’d usually do, but then there are scoped npm packages that mess up this logic.

[rauanmayemir]

Also, esy might eventually switch to scoping all npm packages with @npm/, same as with @opam/.

[rauanmayemir]

What's depending on gmp?

Unless we want to tell users to install it separately, we'll need to depend on https://github.com/esy-packages/esy-gmp. I'll fix it to give us cflags/libs variables.

[wokalski]

@rauanmayemir zarith, a transitive dependency of httpkit.