<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il sab 20 giu 2020, 22:55 Thomas Petazzoni <<a href="mailto:thomas.petazzoni@bootlin.com" target="_blank" rel="noreferrer">thomas.petazzoni@bootlin.com</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, 20 Jun 2020 22:48:55 +0200<br>
Angelo Compagnucci <<a href="mailto:angelo@amarulasolutions.com" rel="noreferrer noreferrer" target="_blank">angelo@amarulasolutions.com</a>> wrote:<br>
<br>
> It's really that important to split it? No problem to do that, but this<br>
> package installs a service and the way it is doing in the new version is<br>
> the one proposed in the patch. The old version could have been different,<br>
> I've not checked btw.<br>
> Anyway, having this is required to have the service correctly installed, so<br>
> I don't think the patch should be splitted because the bump really requires<br>
> it.<br>
<br>
How is this related to the bump? Is it the version bump that adds the<br>
service file? Or did it already exist before the bump?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Cups is a service and a service without an init script is useless.</div><div dir="auto"><br></div><div dir="auto">So, to have the new version working it needs the rcdir change and the systemv script. Systemd script is instead fine.</div><div dir="auto"><br></div><div dir="auto">If you are asking me to even fix the older version I can do that, but I'm doing a bump and fixing at the same time because either the bump will not work.</div><div dir="auto"><br></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">
<br>
> > > +define CUPS_INSTALL_INIT_SYSV<br>
> > > +     @$(RM) $(TARGET_DIR)/etc/init.d/cups  <br>
> ><br>
> > This should also be done in the systemd case I believe, and should not<br>
> > have the @.<br>
> >  <br>
> <br>
> Not for systemd, cause the package installs the .service file and not the<br>
> systemv related ones.<br>
<br>
OK, I see: when --enable-systemd is used to build cups, it doesn't<br>
install its own init script. Makes sense. Thanks for explaining this<br>
aspect!<br>
<br>
Thomas<br>
-- <br>
Thomas Petazzoni, CTO, Bootlin<br>
Embedded Linux and Kernel engineering<br>
<a href="https://bootlin.com" rel="noreferrer noreferrer noreferrer" target="_blank">https://bootlin.com</a><br>
_______________________________________________<br>
buildroot mailing list<br>
<a href="mailto:buildroot@busybox.net" rel="noreferrer noreferrer" target="_blank">buildroot@busybox.net</a><br>
<a href="http://lists.busybox.net/mailman/listinfo/buildroot" rel="noreferrer noreferrer noreferrer" target="_blank">http://lists.busybox.net/mailman/listinfo/buildroot</a><br>
</blockquote></div></div></div>