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

[Robin Roevens <robin.roevens@disroot.org>]

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

Hi all

Had anybody already had a chance to start reviewing this patch-set
during my absence ?

I also have changes to Pakfire in the pipeline to actually use the
metadata provided by this set, but I'm waiting to submit those patches
as they depend on the acceptance of this set (and on my
other patch "pakfire: implement function to parse meta files" to counter Jonathan's comments about de pakfire code quality which I originally duplicated in a previous prototype of the changes to come.)

Regards

Robin

Robin Roevens schreef op do 01-07-2021 om 21:15 [+0200]:
> Hi
> 
> Second attemt to submit this patchset. Hoping the mailserver won't
> find
> malicious URLs in it.
> 
> For completeness, the summary included in the first attempt:
> 
> 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.