schveiguy / iopipe

D language library for modular io
Boost Software License 1.0
77 stars 6 forks source link

Builds failing with v0.2.0 - std.io not retrieved #30

Closed jondegenhardt closed 4 years ago

jondegenhardt commented 4 years ago

Hi Steve,

Congrats on getting the new @safe version out. Good to see progress here.

I ran my weekly cron job on dcat-perf which uses iopipe. It failed, it appears that std.io is no longer automatically pulled as a dependency. The builds are travis-ci.com, here.

The only thing I could find is this commit: Make dependency on io optional.... It added the optional=true flag to the dub.sdl file.

dependency "io" version="~>0.2.4" optional=true

This commit was done a while ago, but from the git tags it appears that v0.2.0 is the first time it got tagged.

Is this something I should be changing in my dub config files (adding std.io as a dependency), or is this a dependency iopipe should take care of?

schveiguy commented 4 years ago

Yes, I forgot I did that. Technically, nothing in iopipe depends on std.io except openDev (which is deprecated) and the example programs.

You should definitely depend on io and then openDev becomes available, but still deprecated (see here: https://github.com/schveiguy/iopipe/blob/d38b289d4527647ecc05f22eccae6f1f79ab3e6d/source/iopipe/stream.d#L10).