[Bug 2173751] Review Request: mod_auth_cas - unretire package

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2173751

Carl George 🤠 <carl@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(carl@xxxxxxxxxx)  |



--- Comment #26 from Carl George 🤠 <carl@xxxxxxxxxx> ---
Indeed, in the current state the patches aren't being applied.  You can do this
with manually with the %patchX macro, but I recommend doing it automatically
with %autosetup instead.  GitHub and git default to creating patches with "a"
and "b" prefixes on the filenames, so you'll need to strip those out by passing
the `-p 1` flag.

 %prep
-%setup -q
+%autosetup -p 1


The last update introduced some tab characters, make sure to switch those to
spaces to avoid this rpmlint warning.

mod_auth_cas.spec:12: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab:
line 12)


-- 
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=2173751
_______________________________________________
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




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux