public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pdm-devel] [PATCH datacenter-manager 0/1] buildsys: initialize pwt-assets as submodule automatically
@ 2025-10-29 14:14 Fiona Ebner
  2025-10-29 14:14 ` [pdm-devel] [PATCH datacenter-manager 1/1] " Fiona Ebner
  0 siblings, 1 reply; 2+ messages in thread
From: Fiona Ebner @ 2025-10-29 14:14 UTC (permalink / raw)
  To: pdm-devel

When the submodule is not initialized there would be some errors at
the beginning of the build like:
> Error: Can't find stylesheet to import.
but then cargo would continue and compile everything only for the
build to fail at the very end.

proxmox-datacenter-manager:

Fiona Ebner (1):
  buildsys: initialize pwt-assets as submodule automatically

 ui/Makefile | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)


Summary over all repositories:
  1 files changed, 4 insertions(+), 1 deletions(-)

-- 
Generated by git-murpp 0.5.0


_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [pdm-devel] [PATCH datacenter-manager 1/1] buildsys: initialize pwt-assets as submodule automatically
  2025-10-29 14:14 [pdm-devel] [PATCH datacenter-manager 0/1] buildsys: initialize pwt-assets as submodule automatically Fiona Ebner
@ 2025-10-29 14:14 ` Fiona Ebner
  0 siblings, 0 replies; 2+ messages in thread
From: Fiona Ebner @ 2025-10-29 14:14 UTC (permalink / raw)
  To: pdm-devel

When the submodule is not initialized there would be some errors at
the beginning of the build like:
> Error: Can't find stylesheet to import.
but then cargo would continue and compile everything only for the
build to fail at the very end.

Copied from Stoiko's equivalent patch for PMG [0].

[0]: https://lore.proxmox.com/pmg-devel/20251028163628.79739-2-s.ivanov@proxmox.com/

Co-developed-by: Stoiko Ivanov <s.ivanov@proxmox.com>
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---

By chance, I stumbled upon Stoiko's patch for PMG after running into
the issue myself in PDM :)

 ui/Makefile | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/ui/Makefile b/ui/Makefile
index 61e5c77..49ae8ee 100644
--- a/ui/Makefile
+++ b/ui/Makefile
@@ -75,8 +75,11 @@ install: $(COMPILED_OUTPUT) index.hbs
 	install -m0644 dist/material-yew-style.css $(DESTDIR)$(UIDIR)
 	install -m0644 dist/desktop-yew-style.css $(DESTDIR)$(UIDIR)
 
+.PHONY: submodule
+submodule:
+	test -f "pwt-assets/README.md" || git submodule update --init
 
-$(BUILDDIR):
+$(BUILDDIR): submodule
 	rm -rf $@ $@.tmp
 	mkdir -p $@.tmp/ui
 	cp -a debian/ src/ pwt-assets/ images/ css/ index.hbs Makefile Cargo.toml $@.tmp/ui
-- 
2.47.3



_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-10-29 14:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-10-29 14:14 [pdm-devel] [PATCH datacenter-manager 0/1] buildsys: initialize pwt-assets as submodule automatically Fiona Ebner
2025-10-29 14:14 ` [pdm-devel] [PATCH datacenter-manager 1/1] " Fiona Ebner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal