<div dir="auto"><div>Hi,<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El dom., 8 mar. 2020 20:25, Carsten Schoenert <<a href="mailto:c.schoenert@t-online.de" rel="noreferrer noreferrer noreferrer noreferrer" target="_blank">c.schoenert@t-online.de</a>> escribió:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
>> I enabled the gitlab CI using the configuration provided by salsa-ci-team<br>
<br>
Some more specific things.<br>
<br>
You have added an override for dh_auto_install but you basically run<br>
just a copy call here. This isn't needed as we have sequencer files that<br>
do this, they are needed to be named [binary-package-name].install. So<br>
instead of having a copy call in debian/rules you should use the<br>
.install file here. For this package here you would need just one line<br>
now. The required folder(s) will get created automatically.</blockquote></div></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> $ cat debian/ocf-spec-core.install <br>
> swagger2.0/*.swagger.json usr/share/ofc/datamodels<br>
<br>
Doing this the override in d/rules is now only containing the chmod<br>
command. And yes, this is somehow needed. But not in the dh_auto_install<br>
step. For fixing permissions there is the helper dh_fixperms used.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><div dir="auto">I'm aware that install.* file exists, but I prefer to stick with the install and fixperms in one point instead of having the install file and fixperms helper. The package didn't have a build system, so i prefer to reflect it in the debian makefile.</div><div dir="auto"><br></div><div dir="auto">If it wasn't need to specify the files in the fixperms override, I had been go for the other way.</div><div dir="auto"><br></div><div dir="auto"><span style="font-family:sans-serif">On the other way, now that I see it, the first chmod expasion is not needed.</span></div></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
But one more thing you need to have in mind here. Some shells on other<br>
platforms can't handle the wildcard sign * and your package would<br>
getting FTBFS than, so you will need to do this differently. Using find<br>
will work on all platforms, so I'd do it this way finally.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div><div dir="auto"><div dir="auto">Glob is not posix/sus v3?</div><div dir="auto"><br></div></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> diff --git a/debian/rules b/debian/rules<br>
> index be0389a..2ab173c 100755<br>
> --- a/debian/rules<br>
> +++ b/debian/rules<br>
> @@ -7,7 +7,6 @@<br>
>  %:<br>
>         dh $@<br>
>  <br>
> -override_dh_auto_install:<br>
> -       mkdir -p debian/ocf-spec-core/usr/share/ofc/datamodels<br>
> -       cp -a swagger2.0/*.swagger.json debian/ocf-spec-core/usr/share/ofc/datamodels<br>
> -       chmod ugo-x swagger2.0/*.swagger.json debian/ocf-spec-core/usr/share/ofc/datamodels/*<br>
> +override_dh_fixperms:<br>
> +       dh_fixperms<br>
> +       find debian/ocf-spec-core/usr/share/ofc/datamodels -type f -exec chmod 644 {} \;</blockquote></div></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
And one more generic suggestion, I'd like to point to DEP 14. Maybe you<br>
already know about this.<br>
<br>
<a href="https://dep-team.pages.debian.net/deps/dep14/" rel="noreferrer noreferrer noreferrer noreferrer noreferrer noreferrer" target="_blank">https://dep-team.pages.debian.net/deps/dep14/</a><br>
<br>
Consider a branch renaming from master to debian/sid. No package upload<br>
did happen yet so it's easy now to squash existing data and do a clean<br>
new upload if you agree. If you have problems doing this I can help out<br>
here of course.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">As is a draft, I prefer to stick with the clasic layout master/upstream/pristine-tar.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>