Skip to content

Make core library no_std#25

Merged
nrc merged 9 commits into
nrc:masterfrom
emberian:no_std
May 4, 2016
Merged

Make core library no_std#25
nrc merged 9 commits into
nrc:masterfrom
emberian:no_std

Conversation

@emberian

@emberian emberian commented May 4, 2016

Copy link
Copy Markdown
Contributor

No description provided.

@emberian

emberian commented May 4, 2016

Copy link
Copy Markdown
Contributor Author

Note that I've removed the version bump and started pushing miscelanneous commits, let me know if you want me to break them out.

Comment thread Cargo.toml

[[bin]]
name = "xmas_elf"
publish = "false"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is required otherwise we publish the bin (which is only meant for testing) to crates.io instead of the lib

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cargo seems to claim it's unused. What does it even mean to publish a bin? The name xmas-elf becomes usable as a cargo install as well?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Also, it's only documented as a field on package sections, not bins)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, the first time I published this to crates.io, it used src/bin/main.rs as the crate (iirc) and it was useless. So I added this line and then publish published the lib (see this commit - 5e2cc78). This may well be horrible cargo culting (no pun intended), I just did what someone on irc told me rather than look into it properly.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about publishing, but I've been using this branch as a git dep and it's been working without issue.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, I'll give it a go

@nrc nrc merged commit 4e7d5d4 into nrc:master May 4, 2016
@nrc

nrc commented May 4, 2016

Copy link
Copy Markdown
Owner

Thank you!

@emberian

emberian commented May 4, 2016

Copy link
Copy Markdown
Contributor Author

Thank you :)

@JIghtuse JIghtuse mentioned this pull request May 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants