https://bugzilla.redhat.com/show_bug.cgi?id=2266544 --- Comment #11 from Fabio Valentini <decathorpe@xxxxxxxxx> --- (In reply to Jens Reimann from comment #10) > Thanks again for the review. I like the idea of having the package named > `trunk`, I'll be working towards that. > > > PS: v0.19.0 was released in the meantime. > > I know, that was me :) Already updated the spec file. Great, thanks! > When it comes to the positions of fields in the file, I had (still have) no > idea what's the preference. I'll just move them around as you suggested. :thumbsup: > I am struggling adding the build flags to other tasks though, adding them > like this: > > ``` > %build > %cargo_build %{_trunk_features} > %{cargo_license_summary} %{_trunk_features} > %{cargo_license} %{_trunk_features} > LICENSE.dependencies > %{cargo_vendor_manifest} %{_trunk_features} > ``` > > I'll run into this: > > ``` > /var/tmp/rpm-tmp.rflLYE: line 58: syntax error near unexpected token `-n' > error: Bad exit status from /var/tmp/rpm-tmp.rflLYE (%build) > ``` > > Maybe it's something obvious that I miss, but I don't see it. I am using the > same way as with the `%cargo_build` (and others). Adding the arguments this way results in them being appended to the expanded macro instead of being treated as arguments to the macro. You need to do move the flags inside the curly braces for them to be treated as the actual macro arguments, like this: ``` > %build %cargo_build %{_trunk_features} %{cargo_license_summary %{_trunk_features}} %{cargo_license %{_trunk_features}} > LICENSE.dependencies %{cargo_vendor_manifest} ``` (And the "%cargo_vendor_manifest" macro does not accept any arguments, so don't pass the flags there.) -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2266544 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202266544%23c11 -- _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue