On Thu, Mar 12, 2020 at 10:42 PM William Roberts <bill.c.roberts@xxxxxxxxx> wrote: > On Thu, Mar 12, 2020 at 3:21 PM Stephen Smalley > <stephen.smalley.work@xxxxxxxxx> wrote: > > > > On Thu, Mar 12, 2020 at 9:59 AM William Roberts > > <bill.c.roberts@xxxxxxxxx> wrote: > > > > > > I will fix the subject and add the Fixes: #165 to the commit when I > > > stage them if thats ok? > > > > Fixes both 165 and 204 IIUC because the real underlying issue for both was the > > fact that he was building with LTO and it was doing the wrong thing > > with hidden_def/hidden_proto. > > Otherwise works for me. You can download the series from patchwork or > > via get-lore-mbox > > to pick up all the Acks automatically. > > I have no idea how to do that, Ill have to spend some time and > familiarize myself with it. You can simply go to https://patchwork.kernel.org/project/selinux/list/, click on any patch in the series (e.g. https://patchwork.kernel.org/patch/11430261/) and copy the address of the "series" button/link in the upper right corner. Then you just do: curl <link> | git am There is certainly a faster way to do it using the Patchwork CLI, but I never bothered to set that up :) > For now I just manually added the acks and did the edits we discussed > and rebased ontop > of current master. Their was a simple conflict in libsepol/src/services.c. > > I have them staged here: > https://github.com/SELinuxProject/selinux/pull/207 The CI failed because libsepol/src/services.c still includes "dso.h" in the rebased commits. You'll need to fix that first. > > And since their were some minor edits before they go in, ill let them > sit a couple > of days for any last objections. Ill merge Monday March 16 end of workday if all > is silent. -- Ondrej Mosnacek <omosnace at redhat dot com> Software Engineer, Security Technologies Red Hat, Inc.
