--
You received this message because you are subscribed to a topic in the Google Groups "autopkg-discuss" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/autopkg-discuss/CwzbfbGx4cU/unsubscribe.
To unsubscribe from this group and all its topics, send an email to autopkg-discu...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
What I am noticing however is that all of the recipes fail to extract and use an icon for the application or package that is they end up in the munki repo with generic icons.
On Mar 12, 2015, at 7:53 AM, John Lockwood <jeloc...@gmail.com> wrote:On a different topic, I am finding that AutoPkgr and/or AutoPkg are not adding icons for items they add to my repo, so the just added OracleJava8 has a generic installer package icon.No AutoPkg recipe (that I know of) adds icons to a Munki repo.
It still is not working and I feel it should.
munkiimport will now offer to attempt to extract an icon for items that don't already have an icon in the repo:
It was this documentation plus the fact that MunkiAdmin does appear to somehow get munkiimport to extract icons - unless that author has in effect duplicated what the munki documentation says munkiimport should already be doing, that led me to believe it should work.
On Mar 17, 2015, at 6:54 AM, John Lockwood wrote:It still is not working and I feel it should.John:I wrote both pieces of code: munkiimport, and the MunkiImporter processor in AutoPkg.THE MUNKIMPORTER PROCESSOR DOES NOT CURRENTLY ATTEMPT TO EXTRACT OR IMPORT ICONS.I'm not sure if I can make that any clearer.-Greg
According to the munkiimport documentation it is supposed to automatically find an icon and extract it or at least try to, this works via MunkiAdmin, but as I keep saying, it does not work via AutoPkg.I can understand that if an existing icon exists with a file name that matches the name in the pkginfo it will then munkiimport does not have to do an icon extraction and Managed Software Center will utilise that existing file, e.g. if the pkginfo name is GoogleChrome then an already existing icon file called GoogleChrome or GoogleChrome.png should work, however as I read the documentation, munkiimport should be examining the package or disk image and automatically if possible extracting an icon to produce said icon file. This is not happening when using AutoPkg. As AutoPkg runs munkiimport and munkiimport is supposed to do this something seems wrong.As a real world example the recipe for Google Chrome uses a pkginfo name of GoogleChrome, however after AutoPkg has run that recipe and added a new Google Chrome to the repo there is no icon file with that name i.e. GoogleChrome or GoogleChrome.png, as this relates to a simple disk image containing an application it should be the easiest possible scenario for munkiimport to handle. This happened when I had no existing GoogleChrome entry in my repo.As a workaround I have just manually added icon files that do match the pkginfo name e.g. GoogleChrome, based on this I would expect that next time GoogleChrome is updated via AutoPkg it should show an icon because the file already exists. This does not address the issue however that when AutoPkg runs munkiimport it is not doing it! That is munkiimport is not extracting an icon for a new entry where an icon file does not yet exist in the repo.
Clearly the Google Chrome application does contain an icon so one is available to extract.
On Tuesday, 17 March 2015 13:39:52 UTC, Greg Neagle wrote:
On Mar 17, 2015, at 3:53 AM, John Lockwood wrote:What I am noticing however is that all of the recipes fail to extract and use an icon for the application or package that is they end up in the munki repo with generic icons.Did you miss my response the last time you asked this question?
My apologies, you wrote the software so you definitely would know.I merely would quote the following from the Munki wiki/documentation and now raise this as a documentation error. See https://code.google.com/p/munki/wiki/ProductIconsmunkiimport
munkiimport will now offer to attempt to extract an icon for items that don't already have an icon in the repo:
It was this documentation plus the fact that MunkiAdmin does appear to somehow get munkiimport to extract icons - unless that author has in effect duplicated what the munki documentation says munkiimport should already be doing, that led me to believe it should work.
On Tuesday, 17 March 2015 13:56:51 UTC, Greg Neagle wrote:On Mar 17, 2015, at 6:54 AM, John Lockwood wrote:It still is not working and I feel it should.John:I wrote both pieces of code: munkiimport, and the MunkiImporter processor in AutoPkg.THE MUNKIMPORTER PROCESSOR DOES NOT CURRENTLY ATTEMPT TO EXTRACT OR IMPORT ICONS.I'm not sure if I can make that any clearer.-GregAccording to the munkiimport documentation it is supposed to automatically find an icon and extract it or at least try to, this works via MunkiAdmin, but as I keep saying, it does not work via AutoPkg.I can understand that if an existing icon exists with a file name that matches the name in the pkginfo it will then munkiimport does not have to do an icon extraction and Managed Software Center will utilise that existing file, e.g. if the pkginfo name is GoogleChrome then an already existing icon file called GoogleChrome or GoogleChrome.png should work, however as I read the documentation, munkiimport should be examining the package or disk image and automatically if possible extracting an icon to produce said icon file. This is not happening when using AutoPkg. As AutoPkg runs munkiimport and munkiimport is supposed to do this something seems wrong.As a real world example the recipe for Google Chrome uses a pkginfo name of GoogleChrome, however after AutoPkg has run that recipe and added a new Google Chrome to the repo there is no icon file with that name i.e. GoogleChrome or GoogleChrome.png, as this relates to a simple disk image containing an application it should be the easiest possible scenario for munkiimport to handle. This happened when I had no existing GoogleChrome entry in my repo.As a workaround I have just manually added icon files that do match the pkginfo name e.g. GoogleChrome, based on this I would expect that next time GoogleChrome is updated via AutoPkg it should show an icon because the file already exists. This does not address the issue however that when AutoPkg runs munkiimport it is not doing it! That is munkiimport is not extracting an icon for a new entry where an icon file does not yet exist in the repo.Clearly the Google Chrome application does contain an icon so one is available to extract.
On Tuesday, 17 March 2015 13:39:52 UTC, Greg Neagle wrote:On Mar 17, 2015, at 3:53 AM, John Lockwood wrote:What I am noticing however is that all of the recipes fail to extract and use an icon for the application or package that is they end up in the munki repo with generic icons.Did you miss my response the last time you asked this question?On Mar 12, 2015, at 7:53 AM, John Lockwood wrote:On a different topic, I am finding that AutoPkgr and/or AutoPkg are not adding icons for items they add to my repo, so the just added OracleJava8 has a generic installer package icon.No AutoPkg recipe (that I know of) adds icons to a Munki repo.There currently is no AutoPkg processor that extracts and imports icons into a Munki repo. Since icons don't change that often from release to release, it has not been a priority. When I import a new version of Chrome, it just uses a previously imported icon.Could such a processor be written and incorporated into Munki recipes? Sure.-Greg
Thinking further, you seem to be implying that the (AutoPkg) munkiimport processor and the (Munki) munkiimport are different pieces of code. The fact that AutoPkg requires the installation of the munki tools including munkiimport implied that AutoPkg was directly or indirectly calling the (Munki) version of munkiimport which according the documentation should be able to extract icons.If there is an intermediary between AutoPkg and munki's munkiimport i.e. the 'munkiimport processor' then I would request that it be 'enhanced' to import icons if possible.
Having practically identical names for two different munkiimport pieces of code clearly has led to confusion. Perhaps renaming the AutoPkg one 'munkiimportprocessor' all one word, would help.
I was aware you wrote both AutoPkg and Munki and this made the expectation that they were one and the same thing even more logical.