-
Notifications
You must be signed in to change notification settings - Fork 186
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.
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
Update dependencies. #547
Update dependencies. #547
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,13 +15,13 @@ workspace = ".." | |
|
||
[dependencies] | ||
ash = { path = "../ash", version = "0.35", default-features = false } | ||
raw-window-handle = "0.3" | ||
raw-window-handle = "0.4.2" | ||
|
||
[target.'cfg(any(target_os = "macos", target_os = "ios"))'.dependencies] | ||
raw-window-metal = "0.1" | ||
raw-window-metal = "0.2.0" | ||
|
||
[dev-dependencies] | ||
winit = "0.19.4" | ||
winit = "0.26.1" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This probably needs additional changes, no? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. cargo check --workspace seemed to work There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
You should however also |
||
ash = { path = "../ash", version = "0.35", default-features = false, features = ["linked"] } | ||
|
||
[[example]] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,8 +5,8 @@ authors = ["maik klein <maikklein@googlemail.com>"] | |
edition = "2018" | ||
|
||
[dependencies] | ||
winit = "0.25.0" | ||
image = "0.10.4" | ||
winit = "0.26.1" | ||
image = "0.23.14" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we need these minimum patch versions? Otherwise just omit them entirely, this is just an example. |
||
# The examples require the validation layers, which means the SDK or | ||
# equivalent development packages should be present, so we can link | ||
# directly and benefit from the infallible `Entry` constructor. | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,15 +5,15 @@ authors = ["Maik Klein <maikklein@googlemail.com>"] | |
edition = "2018" | ||
|
||
[dependencies] | ||
bindgen = "0.58" | ||
heck = "0.3" | ||
bindgen = "0.59.2" | ||
heck = "0.4.0" | ||
itertools = "0.10" | ||
nom = "6.0" | ||
once_cell = "1.7" | ||
once_cell = "1.9.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Again no need for patch versions. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. that's a bug in cargo upgrade --skip-compatible There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd do these by hand :) |
||
proc-macro2 = "1.0" | ||
quote = "1.0" | ||
regex = "1.4" | ||
vk-parse = { version = "0.6", features = ["vkxml-convert"] } | ||
regex = "1.5.4" | ||
vk-parse = { version = "0.7.0", features = ["vkxml-convert"] } | ||
vkxml = "0.3" | ||
|
||
[dependencies.syn] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
#![recursion_limit = "256"] | ||
|
||
use heck::{CamelCase, ShoutySnakeCase, SnakeCase}; | ||
use heck::{ToLowerCamelCase, ToShoutySnakeCase, ToSnakeCase}; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we use the cleaner EDIT: We can't in the places of |
||
use itertools::Itertools; | ||
use nom::{ | ||
alt, char, | ||
|
@@ -1200,7 +1200,7 @@ pub fn generate_extension_commands<'a>( | |
|
||
let ident = format_ident!( | ||
"{}Fn", | ||
extension_name.to_camel_case().strip_prefix("Vk").unwrap() | ||
extension_name.to_lower_camel_case().strip_prefix("Vk").unwrap() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lower... So this won't strip the uppercase Vk prefix if the whole thing is lowered or will it retain characters that were uppercase? Please run the generator at least once to make sure output doesn't change or remains what we expect. |
||
); | ||
let fp = generate_function_pointers(ident.clone(), &commands, &aliases, fn_cache); | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#505