Hi Sophia.... thanks for submitting this. In general it looks good but there are a couple things:
1: Why is there a separate whitelist for this? Does this need it's own whitelist, or would this be in a general certification remote whitelist? If that's the case, the whitelist should simply be called opencompute-certify-remote.whitelist.
2: if you want the config file me.cfg to be installed, you need to add it to the approrpiate files in the debian directory. here are teh places where the examples directory is referenced:
You're probably interested in checkbox.install and checkbox.postinst
Without adding it to those files, the config file will not be installed with the package.
3: in the ipmi_sel_entries script, you reference ../examples/me.cfg twice. The proper installed location is /etc/checkbox.d/ (See 2 above).
4: Finally, you also need to update debian/changelog and add an entry under your name that briefly describes the changes you've made. See other entries in the changelog for examples.
Fix those few items (and just let me know more about item 1) and I'll push this through.
Hi Sophia.... thanks for submitting this. In general it looks good but there are a couple things:
1: Why is there a separate whitelist for this? Does this need it's own whitelist, or would this be in a general certification remote whitelist? If that's the case, the whitelist should simply be called opencompute- certify- remote. whitelist.
2: if you want the config file me.cfg to be installed, you need to add it to the approrpiate files in the debian directory. here are teh places where the examples directory is referenced:
debian/ checkbox- hw-collection. install: usr/share/ checkbox/ examples/ checkbox- hw-collection. ini checkbox. install: usr/share/ checkbox/ examples/ checkbox. ini checkbox. install: usr/share/ checkbox/ examples/ network. cfg checkbox. install: usr/share/ checkbox/ examples/ virtualization. cfg checkbox. install: usr/share/ checkbox/ examples/ org.freedesktop .policykit. checkbox. policy usr/share/ polkit- 1/actions/ checkbox- ocp-cli. install: usr/share/ checkbox/ examples/ checkbox- ocp-cli. ini checkbox- ocp-gtk. install: usr/share/ checkbox/ examples/ checkbox- ocp-gtk. ini checkbox- ocp-qt. install: usr/share/ checkbox/ examples/ checkbox- ocp-qt. ini checkbox- ocp-urwid. install: usr/share/ checkbox/ examples/ checkbox- ocp-urwid. ini checkbox. postinst: cp /usr/share/ checkbox/ examples/ network. cfg /etc/checkbox.d/ checkbox. postinst: cp /usr/share/ checkbox/ examples/ virtualization. cfg /etc/checkbox.d/
debian/
debian/
debian/
debian/
debian/
debian/
debian/
debian/
debian/
debian/
You're probably interested in checkbox.install and checkbox.postinst
Without adding it to those files, the config file will not be installed with the package.
3: in the ipmi_sel_entries script, you reference ../examples/me.cfg twice. The proper installed location is /etc/checkbox.d/ (See 2 above).
4: Finally, you also need to update debian/changelog and add an entry under your name that briefly describes the changes you've made. See other entries in the changelog for examples.
Fix those few items (and just let me know more about item 1) and I'll push this through.
Thanks!