build.func VLAN

[DrOwl]

✅ Have you read and understood the above guidelines?

yes

📜 What is the name of the script you are using, or where do you have Problems?

build.func

📝 Provide a clear and concise description of the issue.

Hello,
I am running the dev version of Frigate install on a Proxmox 9 server.
I use VLAN's to am running the advanced installer
There appears to be a bug when creating the net_string

NET_STRING+="${VLAN:+,tag=$VLAN}"

In my case this creates a string with ",tag=,tag=100" as "$VLAN" already contains ",tag=X"

I tested with a local mod to build.func and install succeeded

NET_STRING+="${VLAN:+$VLAN}"

🔄 Steps to reproduce the issue.

Install a CT using advanced option choosing to set a vlan.

❌ Paste the full error output (if available).

+ pct create 202 local:vztmpl/debian-12-standard_12.12-1_amd64.tar.zst -features nesting=1 -hostname frigate -tags nvr -net0 name=eth0,bridge=vmbr1,ip=dhcp,tag=,tag=100,ip6=auto -onboot 1 -cores 4 -memory 4096 -unprivileged 0 -rootfs disks:20

🖼️ Additional context (optional).

No response

[MickLesk]

the complete core changes in this repo. You should use default while testing. Ive changed over 3000 Lines atm

[DrOwl]

I used this dev repo as I was wanting to test the Frigate 16 install.
I use vlan's and the install fails if the correct one is not chosen, guess I could try quickly swap it manually.

Very impressed with the huge amount of work of work that is being done things like fetch_and_deploy_gh_release are a game changer

[MickLesk]

It wasn't actually my goal today to fix the issue, but you can try again.

[DrOwl]

Thank you for the quick work, that Fixed the vlan issue for me.