Skip to content

Embed Windows application manifests - #10878

Merged
bors merged 2 commits into
rust-lang:masterfrom
vadimcn:manifest
Dec 13, 2013
Merged

Embed Windows application manifests#10878
bors merged 2 commits into
rust-lang:masterfrom
vadimcn:manifest

Conversation

@vadimcn

Copy link
Copy Markdown
Contributor

In order to avoid UAC installer detection silliness...

Closes#10512

Comment threadsrc/librustc/back/link.rs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it possible to just do this as a literal:

"<?xml version='1.0' encoding='UTF-8' standalone='yes'?>\r<assembly xmlns='urn:schemas-microsoft-com:asm.v1' manifestVersion='1.0'>\r <trustInfo xmlns='urn:schemas-microsoft-com:asm.v3'>\r..."

?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

concat!() performs compile-time concatenation, so this is a single literal.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I know that, I was talking at a source level because it just seems a little peculiar to have 10 strings rather than 1 (and concat! will possibly be removed, because it is designed to allow things like concat!("foo", some_macro!()) rather than just plain strings, and this "local expansion" can cause problems with hygiene, apparently.)

@alexcrichton

Copy link
Copy Markdown
Member

silly windows, but nice work!

@vadimcn

Copy link
Copy Markdown
ContributorAuthor

@alexcrichton: moved manifest stuff to a separate file, added comments as you asked. r?

@alexcrichton

Copy link
Copy Markdown
Member

Needs a rebase.

@vadimcn

Copy link
Copy Markdown
ContributorAuthor

@alexcrichton: Rebased

@alexcrichton

Copy link
Copy Markdown
Member

It appears some rebasing indentation has gone wrong. While you're at it, could you rebase these into a single commit?

@vadimcn

Copy link
Copy Markdown
ContributorAuthor

Done.

@vadimcn

Copy link
Copy Markdown
ContributorAuthor

@alexcrichton: retry

...please :)

@vadimcn

Copy link
Copy Markdown
ContributorAuthor

Grrr... this trailing whitespace check is infuriating.
BTW, tidy seems to be broken on Windows, so make check did not catch this.

Also, rebased

@vadimcn

Copy link
Copy Markdown
ContributorAuthor

I can't fathom how could this have broken Android but not other unixes... Perhaps a spurious build machine error?

bors added a commit that referenced this pull request Dec 13, 2013
In order to avoid UAC installer detection silliness...
Closes#10512
@borsbors closed this Dec 13, 2013
@bors
bors merged commit d4d1310 into rust-lang:masterDec 13, 2013
@vadimcn
vadimcn deleted the manifest branch March 1, 2015 08:04
flip1995 pushed a commit to flip1995/rust that referenced this pull request May 17, 2024
…et,GuillaumeGomez
Allow more attributes in `clippy::useless_attribute`
Fixesrust-lang#12753Fixesrust-lang#4467Fixesrust-lang#11595Fixesrust-lang#10878
changelog: [`useless_attribute`]: Attributes allowed on `use` items now include `ambiguous_glob_exports`, `hidden_glob_reexports`, `dead_code`, `unused_braces`, and `clippy::disallowed_types`.
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
10951: feat: assist to generate documentation templates r=Veykril a=numero-744
Closesrust-lang#10878, rust-lang#1915 and rust-lang#4767
Full description is in rust-lang#10878, better read [the tests](https://github.com/rust-analyzer/rust-analyzer/pull/10951/files#diff-7a64e2efb66b2625443340fcbc96d531baff12c17cc0aaf51885ea94f67de254R424) to understand what this feature does.
- [x] There is one remaining thing about non-`pub` functions, what do you think about it?
- [x] In this PR [empty examples are generated](https://github.com/rust-analyzer/rust-analyzer/pull/10951/files#diff-7a64e2efb66b2625443340fcbc96d531baff12c17cc0aaf51885ea94f67de254R99) for `trait` functions, but maybe no examples should be provided at all.
- [x] If there is already a documentation, add another one with a separator ([currently done](https://github.com/rust-analyzer/rust-analyzer/pull/10951/files#diff-7a64e2efb66b2625443340fcbc96d531baff12c17cc0aaf51885ea94f67de254R74)) or simply disable this assist?
- [x] I will check once more that the generated examples are correct (ie. they are easy to fill before that they are built and tested)
Comments appreciated 😄 Co-authored-by: Côme ALLART <come.allart@etu.emse.fr>
Sign up for freeto 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.

Windows: rustc should embed application manifest into executables

4 participants

@vadimcn@alexcrichton@huonw@bors