Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Creation of a new spk fails to trigger build #6284

Closed
1 task done
th0ma7 opened this issue Oct 16, 2024 · 1 comment · Fixed by #6282
Closed
1 task done

Creation of a new spk fails to trigger build #6284

th0ma7 opened this issue Oct 16, 2024 · 1 comment · Fixed by #6282
Assignees
@mreid-tt @hgy59
Labels
bug framework

Comments

@th0ma7
Copy link
Contributor

th0ma7 commented Oct 16, 2024 •
edited
Loading

Is this a new Bug?

  • I checked that the bug hasn't been reported before

Package Name

synocli-wheels

What happened?

Relates to #6282 where new package synocli-wheels does not get trigered upon build.

Suspecting an oversight with recent changes to github-action.
@th0ma7 th0ma7 added the bug label Oct 16, 2024
@hgy59
Copy link
Contributor

hgy59 commented Oct 16, 2024

This already popped up latelty. I guess it could be an issue when creating the dependency list with bash variables.
variables are limitted to 1024 chars so dependencies could be lost...
should be addressed in #6255.

@hgy59 hgy59 mentioned this issue Nov 18, 2024
Merged
20 tasks
hgy59 added a commit to th0ma7/spksrc that referenced this issue Nov 26, 2024
@hgy59
update github build action
2b552da 
- add evaluation of python313 dependent packages
- evaluate packages to build and changed dependency folders in build.yml
  to avoid overflow of bash variables (limited to 1024 chars) (fixes SynoCommunity#6284)
hgy59 added a commit to th0ma7/spksrc that referenced this issue Nov 26, 2024
@hgy59
real fix for SynoCommunity#6284
2f537de 
- python dependent packages with name sorted after python31* where removed from packages to build
  (in this context: python-wheels and rdiff-backup)
th0ma7 pushed a commit to th0ma7/spksrc that referenced this issue Jan 7, 2025
@hgy59 @th0ma7
update github build action
a87c435 
- add evaluation of python313 dependent packages
- evaluate packages to build and changed dependency folders in build.yml
  to avoid overflow of bash variables (limited to 1024 chars) (fixes SynoCommunity#6284)
th0ma7 pushed a commit to th0ma7/spksrc that referenced this issue Jan 7, 2025
@hgy59 @th0ma7
real fix for SynoCommunity#6284
15ce51e 
- python dependent packages with name sorted after python31* where removed from packages to build
  (in this context: python-wheels and rdiff-backup)
th0ma7 pushed a commit to th0ma7/spksrc that referenced this issue Jan 9, 2025
@hgy59 @th0ma7
update github build action
4d8e8d2 
- add evaluation of python313 dependent packages
- evaluate packages to build and changed dependency folders in build.yml
  to avoid overflow of bash variables (limited to 1024 chars) (fixes SynoCommunity#6284)
th0ma7 pushed a commit to th0ma7/spksrc that referenced this issue Jan 9, 2025
@hgy59 @th0ma7
real fix for SynoCommunity#6284
819e915 
- python dependent packages with name sorted after python31* where removed from packages to build
  (in this context: python-wheels and rdiff-backup)
@th0ma7 th0ma7 closed this as completed in 8aab04e Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mreid-tt mreid-tt

@hgy59 hgy59

Labels
bug framework
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

dynamic crossenv + python3*-wheels + python310-313 updates th0ma7/spksrc
3 participants
@mreid-tt @hgy59 @th0ma7