-
Notifications
You must be signed in to change notification settings - Fork 345
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
T6527: add legacy Vyatta interpreter files still in use #3745
Conversation
0c09aac
to
731a5c6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should add license headers with attribution to Vyatta at least to all non-trivial files. Maybe even just add them to all files, even allowed-op and similar, just to make it clear where they came from and who the real author is — it's unlikely that we'll get into legal trouble for not attributing trivial files, but I think it's a good thing to do.
We can also clean files from legacy stuff like serial WAN, since it was never supported in VyOS anyway.
src/opt/vyatta/share/vyatta-op/templates/clear/interfaces/connection/node.tag/node.def
Outdated
Show resolved
Hide resolved
src/opt/vyatta/share/vyatta-op/templates/clear/interfaces/connection/node.tag/node.def
Outdated
Show resolved
Hide resolved
src/opt/vyatta/share/vyatta-op/templates/clear/interfaces/connection/node.tag/node.def
Outdated
Show resolved
Hide resolved
All requested changes addressed. |
👍 VyOS CLI smoketests finished successfully! |
@Mergifyio backport circinus |
✅ Backports have been created
|
Change Summary
Remove legacy packages from ISO build.
Are no longer required and migrated to vyos-1y. vyatta-wanloadbalance, vyatta-bash and vyatta-cfg are now a direct dependency of vyos-1x, so everything in one place.
Types of changes
Related Task(s)
Related PR(s)
Smoketest result
Checklist: