Skip to content

Add TARGET environment variable to action.yml#61

Closed
Zerogiven wants to merge 25 commits into
openwrt:mainfrom
Zerogiven:main
Closed

Add TARGET environment variable to action.yml#61
Zerogiven wants to merge 25 commits into
openwrt:mainfrom
Zerogiven:main

Conversation

@Zerogiven
Copy link
Copy Markdown

This is a very hacky solution. Hopefully someone else have a nicier idea how to solve it. Problem is if we set an empty TARGET env variable it uses the default from setup.sh (x86/64).

But for example if you need to build a kernel module especially for mediatek-filogic you have to change the TARGET because it uses "bcm27xx/bcm2710" by default. (no idea why not mediatek/filogic).

With this really bad patch you can overwrite the TARGET

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Refactor docker run commands based on TARGET environment variable.

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Removed conditional check and modification for llvm.download-ci-llvm in Rust Makefile.

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Add patch to silence missing prototypes warnings

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Removed wget command for downloading a patch file.

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
List contents of ltq-adsl directory and display Makefile.

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Comment out ls and cat commands, modify sed command to include additional flags.

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Added sed commands to modify Makefile for kernel package build process.

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Add neutralization steps for KernelPackage in Makefile

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
Removed commented-out lines related to kernel package compilation and Makefile modifications.

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
@aparcar
Copy link
Copy Markdown
Member

aparcar commented Mar 14, 2026

Kindly cleanup this commit history...

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
@aparcar
Copy link
Copy Markdown
Member

aparcar commented Apr 29, 2026

I reworked the docker action to create the right tags, could you please check if this is still needed?

@Zerogiven
Copy link
Copy Markdown
Author

Zerogiven commented May 3, 2026

Ah sorry, totally forgot to answer here because of busy development at work..

And yes, personally i still need it for my automated package builds, mostly for kmod packages - at the moment i am using a custom fork but would be great to come back to official action 🙂 i'll check it out very soon

Thanks for your patience

Signed-off-by: CSoellinger <Zerogiven@users.noreply.github.com>
@Zerogiven
Copy link
Copy Markdown
Author

Yes this worked thx 👍

@Zerogiven Zerogiven closed this May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants