All of lore.kernel.org
 help / color / mirror / Atom feed
diff for duplicates of <21393.1709879703135556993@lists.openembedded.org>

diff --git a/a/1.txt b/N1/1.txt
index aee59ec..c9bb912 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -4,10 +4,14 @@ Two things:
 
 First, I had a patch that I forgot to add to this, where I make the ydotoold.service be installed as a system service instead of a user service, otherwise the rootfs recipe fails as it can't enable it. I'd like suggestions to what you feel better. Sorry for missing that. If there's a more generic option for systemd user services, let me know.  
 One option is: have a patch that whenever we have systemd as a dependency, we install it as system service and not user service.  
-Another option is: disable the service by default, with that I can disable the hard dependency as mentioned above (seems to work just fine)
+Another option is: disable the service by default, with that I can remove the hard dependency as mentioned above (seems to work just fine)
 
 > this is an unterminated string, bitbake would have told you during
 > parsing. Please test the bits before sending for review.
 > It saves time.
 
 I have no idea how this went through as last time I was only changing the packagegroup, sorry.
+
+---- 
+Edit: 
+I'm sending a new patch version (v5) applying the second option, just tested, and it works fine, the only downside is the need to manually having to enable the service. If we disagree, I can implement the fixes on top of that one.
diff --git a/a/2.bin b/N1/2.bin
index d021651..ef226e6 100644
--- a/a/2.bin
+++ b/N1/2.bin
@@ -4,7 +4,7 @@
 
 <p>First, I had a patch that I forgot to add to this, where I make the ydotoold.service be installed as a system service instead of a user service, otherwise the rootfs recipe fails as it can't enable it. I'd like suggestions to what you feel better. Sorry for missing that. If there's a more generic option for systemd user services, let me know.<br>
 One option is: have a patch that whenever we have systemd as a dependency, we install it as system service and not user service.<br>
-Another option is: disable the service by default, with that I can disable the hard dependency as mentioned above (seems to work just fine)</p>
+Another option is: disable the service by default, with that I can remove the hard dependency as mentioned above (seems to work just fine)</p>
 
 <blockquote>
 <p>this is an unterminated string, bitbake would have told you during
@@ -13,3 +13,8 @@ It saves time.</p>
 </blockquote>
 
 <p>I have no idea how this went through as last time I was only changing the packagegroup, sorry.</p>
+
+<hr>
+
+<p>Edit:
+I'm sending a new patch version (v5) applying the second option, just tested, and it works fine, the only downside is the need to manually having to enable the service. If we disagree, I can implement the fixes on top of that one.</p>
diff --git a/a/2.hdr b/N1/2.hdr
index 24bd2b6..6a9de1e 100644
--- a/a/2.hdr
+++ b/N1/2.hdr
@@ -1,2 +1,2 @@
-Content-Type: text/html; charset="utf-8"
+Content-Type: text/html; charset=UTF-8
 Content-Transfer-Encoding: quoted-printable
diff --git a/a/content_digest b/N1/content_digest
index 1f3b825..4fbb09b 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -1,7 +1,7 @@
  "ref\0CAMKF1sq8qy4zA53gTgFq2FZpZ9zvbNjpPY3DnycOVN=yFvW2xA@mail.gmail.com\0"
  "From\0Andr\303\251 Paiusco <andre.paiusco@gmail.com>\0"
  "Subject\0Re: [PATCH v4] ydotool: Add new package\0"
- "Date\0Thu, 07 Mar 2024 22:35:03 -0800\0"
+ "Date\0Thu, 07 Mar 2024 23:07:17 -0800\0"
  "To\0openembedded-devel@lists.openembedded.org\0"
  "\01:1\0"
  "b\0"
@@ -11,13 +11,17 @@
  "\n"
  "First, I had a patch that I forgot to add to this, where I make the ydotoold.service be installed as a system service instead of a user service, otherwise the rootfs recipe fails as it can't enable it. I'd like suggestions to what you feel better. Sorry for missing that. If there's a more generic option for systemd user services, let me know.  \n"
  "One option is: have a patch that whenever we have systemd as a dependency, we install it as system service and not user service.  \n"
- "Another option is: disable the service by default, with that I can disable the hard dependency as mentioned above (seems to work just fine)\n"
+ "Another option is: disable the service by default, with that I can remove the hard dependency as mentioned above (seems to work just fine)\n"
  "\n"
  "> this is an unterminated string, bitbake would have told you during\n"
  "> parsing. Please test the bits before sending for review.\n"
  "> It saves time.\n"
  "\n"
- I have no idea how this went through as last time I was only changing the packagegroup, sorry.
+ "I have no idea how this went through as last time I was only changing the packagegroup, sorry.\n"
+ "\n"
+ "---- \n"
+ "Edit: \n"
+ I'm sending a new patch version (v5) applying the second option, just tested, and it works fine, the only downside is the need to manually having to enable the service. If we disagree, I can implement the fixes on top of that one.
  "\01:2\0"
  "b\0"
  "<p>Hello,</p>\r\n"
@@ -26,7 +30,7 @@
  "\r\n"
  "<p>First, I had a patch that I forgot to add to this, where I make the ydotoold.service be installed as a system service instead of a user service, otherwise the rootfs recipe fails as it can't enable it. I'd like suggestions to what you feel better. Sorry for missing that. If there's a more generic option for systemd user services, let me know.<br>\r\n"
  "One option is: have a patch that whenever we have systemd as a dependency, we install it as system service and not user service.<br>\r\n"
- "Another option is: disable the service by default, with that I can disable the hard dependency as mentioned above (seems to work just fine)</p>\r\n"
+ "Another option is: disable the service by default, with that I can remove the hard dependency as mentioned above (seems to work just fine)</p>\r\n"
  "\r\n"
  "<blockquote>\r\n"
  "<p>this is an unterminated string, bitbake would have told you during\r\n"
@@ -35,5 +39,10 @@
  "</blockquote>\r\n"
  "\r\n"
  "<p>I have no idea how this went through as last time I was only changing the packagegroup, sorry.</p>\r\n"
+ "\r\n"
+ "<hr>\r\n"
+ "\r\n"
+ "<p>Edit:\r\n"
+ "I'm sending a new patch version (v5) applying the second option, just tested, and it works fine, the only downside is the need to manually having to enable the service. If we disagree, I can implement the fixes on top of that one.</p>\r\n"
 
-cd9aea05c56e66ab85c88ec037bff0295c23a5727f92bfb3ce9d2232e9a4def0
+4921c4d7a952d4297f49a178cbdbdc48daaa257ddbe53a2fd3125f33465019cb

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.