|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
ignatenkobrain commented 7 years ago | ||
mich181189 commented 7 years ago Ah hehe - this is actually because I was using this like: rust2rpm -u rust-sdl.spec and in that case args.crate actually contains the spec file name - then we pull the crate out of the spec file. I did it this way precisely because it takes a lot of parsing to get it right otherwise, and I didn't really want to have to get the crate if the version hasn't changed | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
ignatenkobrain commented 7 years ago that's not enough for update, you should always update (Build)Requires as well | ||
mich181189 commented 7 years ago I thought about that, but sometimes I've patched the requires lines so I figured it would make more sense to not touch them (as a first pass) Maybe it should apply all the patches then parse out the requires from the patched crate... | ||
|
||
|
||
|
||
|
||
|
||
|
||
ignatenkobrain commented 7 years ago and this exits unconditionally :) | ||
mich181189 commented 7 years ago Gah fail! I must admit when I wrote this line I wondered if I had got the indentation right! Damn significant whitespace language! ;-) | ||
|
||
|
||
|
||
|
||
Feel free to reject this if you have something else in mind.
This commit adds a new option - -u/--update which when set and a spec file
is passed in, parses out the version and crate from the spec file, and checks
for updates, modifying the spec file if needed to update the package.
It does not change any dependencies since these may have been manually
modified.
that's not always like this.... sometimes it's
rust-%crate
, sometimes it's%crate