Re: [PATCH 0/2] buildprocess: additional pak metadata

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 3613 bytes --]

Hello Robin,

I raised this with Peter, but it looks he didn’t find time to have a look at it yet, or it is a bit more complicated than a one-liner.

Could you try to submit this again, and if it doesn’t work try sending them to my personal email address so that we can keep this going while you are away? I would like to keep the ball rolling…

Best,
-Michael

> On 1 Jul 2021, at 09:29, Robin Roevens <robin.roevens(a)disroot.org> wrote:
> 
> Hi
> 
> Any news on this? Or should I try to submit the patch straight to git ? Or just have a bit more patience?
> (I will be on vacation starting tomorrow for about a month..so I won't be able to resubmit it then until next month)
> 
> Robin
> 
> Michael Tremer schreef op za 26-06-2021 om 13:09 [+0100]:
>> Hello,
>> 
>>> On 25 Jun 2021, at 00:04, Robin Roevens <
>>> robin.roevens(a)disroot.org
>>> > wrote:
>>> 
>>> It seems patch 2/2 of this set is rejected by the mailserver:
>>> 
>>> 554 5.7.1 Rejected due to policy violation: Contains blacklisted URL.
>> 
>> Yes, our mail server seems to do that a lot recently.
>> 
>>> For as far as I can see, the patch does not contain any URL's of any
>>> sort.
>>> 
>>> How should I proceed from here? Is there an alternative way to submit
>>> this patch-set? Or can it be checked what triggers this mailserver
>>> error ?
>> 
>> You have done the right thing by copying postmaster.
>> 
>> Peter and I had a brief discussion about this yesterday. Let’s see what he says after looking at some logs.
>> 
>> Best,
>> -Michael
>> 
>>> 
>>> Regards
>>> Robin
>>> 
>>> Robin Roevens schreef op vr 25-06-2021 om 00:50 [+0200]:
>>>> Hi
>>>> 
>>>> As discussed earlier, I hereby submit a patchset adding extra metadata 
>>>> to all pak's.
>>>> 
>>>> First patch adds the new metadata fields "Summary" and "Services" to
>>>> the 
>>>> meta-file templates and introduces the new macro INSTALL_INITSCRIPTS 
>>>> accepting a space seperated list of initscripts to install to avoid
>>>> duplicating the list of service initscripts. (Once in the new SERVICES
>>>> meta-data field and once by calling INSTALL_INITSCRIPT for each of 
>>>> them).
>>>> The original INSTALL_INITSCRIPT macro is kept (and called by the new
>>>> macro) for corner cases where non-service initscripts need to be 
>>>> installed and for use by non-pak lfs files as they currently don't have
>>>> a SERVICES variable. 
>>>> 
>>>> The second patch adds the new metadata for all pak's in their
>>>> respective
>>>> lfs files. 
>>>> As I went over all pak lfs files, I did not encounter any corner cases
>>>> hence all calls to INSTALL_INITSCRIPT are replaced by calls to the new
>>>> INSTALL_INITSCRIPTS passing the SERVICES variable as argument.
>>>> The only special case maybe worth mentioning is Icinga, where a service
>>>> initscript is installed by a make rule of the source. Hence no call to
>>>> INSTALL_INITSCRIPT or INSTALL_INITSCRIPTS is required. But the service
>>>> is included in the SERVICES variable to have it recorded in the meta-
>>>> file.
>>>> 
>>>> This set does not yet contain changes in pakfire or services.cgi to
>>>> actually do something with the new meta-data.
>>>> Those changes will be posted shortly.
>>>> 
>>>> Regards
>>>> 
>>>> Robin
>>>> 
>>>> 
>>> 
>>> 
>>> -- 
>>> Dit bericht is gescanned op virussen en andere gevaarlijke
>>> inhoud door MailScanner en lijkt schoon te zijn.
>>> 
>> 
>> 
> 
> -- 
> Dit bericht is gescanned op virussen en andere gevaarlijke 
> inhoud door MailScanner en lijkt schoon te zijn.