Skip to content

Add warnings and errors when the crate 'link' attribute looks malformed #614

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
Closed
brson opened this issue Jul 2, 2011 · 0 comments
Closed

Add warnings and errors when the crate 'link' attribute looks malformed #614

brson opened this issue Jul 2, 2011 · 0 comments
Assignees
@brson
Labels
A-linkage Area: linking into static, shared libraries and binaries E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.

Comments

@brson
Copy link
Contributor

brson commented Jul 2, 2011

The link attribute has special meaning on crates, and certain rules should be enforced like:

  • Warn if there is no 'name' or 'vers' item for shared libraries
  • Error if there are two items with the same identifier

Some of this used to exist until I lost it.
@ghost ghost assigned brson Jul 2, 2011
brson added a commit that referenced this issue Jul 5, 2011
@brson
Error if the link attribute has duplicate items. Issue #614
6f8bc40
@brson brson closed this as completed in 10a4737 Jul 5, 2011
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
@bors
Auto merge of rust-lang#614 - Razican:version_update, r=alexcrichton
2015cf1 
Bumped version number

Bumped version number so that the latest changes go to crates.io.
pdietl pushed a commit to pdietl/rust that referenced this issue Apr 23, 2020
@ehuss
Merge pull request rust-lang#614 from bjorn3/patch-1
6d1d6ee 
Fix spell error in attributes/diagnostics.md
ZuseZ4 pushed a commit to EnzymeAD/rust that referenced this issue Mar 7, 2023
@wsmoses
Improve Activity Analysis (rust-lang#614)
aad739c 
* Improve activity analysis

* Fix hypothesis

* Fix doubleload

* Fix tests
GuillaumeGomez pushed a commit to GuillaumeGomez/rust that referenced this issue Apr 18, 2025
@antoyo
Merge pull request rust-lang#614 from Kobzol/ci-success
3e62ac7 
Add CI success job
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brson brson

Labels
A-linkage Area: linking into static, shared libraries and binaries E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@brson