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.

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

[dsm6 branch] DSM 6.1 toolchains #2727

Merged
merged 1 commit into from
Jul 27, 2017

Conversation

ymartin59
Copy link
Contributor

Each one tested by building nano package

@ymartin59 ymartin59 changed the title WIP: DSM 6.1 toolchains WIP: [dsm6 branch] DSM 6.1 toolchains Apr 3, 2017
@ymartin59 ymartin59 force-pushed the dsm6-toolchains-6.1 branch 2 times, most recently from 9f0888e to 975dbe1 Compare April 3, 2017 21:03
@ymartin59 ymartin59 changed the title WIP: [dsm6 branch] DSM 6.1 toolchains [dsm6 branch] DSM 6.1 toolchains Apr 3, 2017
@amyreese
Copy link

I was able to successfully build a package from this and run it on my braswell unit. Thanks for posting this! 👍

@michailf
Copy link
Contributor

TC_FIRMWARE = 6.1-15047-1

Should the "-1" suffix be removed?

@ymartin59
Copy link
Contributor Author

@michailf You may be right concerning firmware version to specify in toolchain but do you know any guideline or documentation describing exactly how to set it and from which source ?

@michailf
Copy link
Contributor

michailf commented May 2, 2017

@ymartin59 No, I do not know of any guidelines. I only observed how TC_FIRMWARE is set in other toolchains. The last number is an update level, the one before is a build number which is unique across different DSM releases.

TC_FIRMWARE appears to be used only for display purposes at this point so it's not that big of a deal.

@m4tt075
Copy link
Contributor

m4tt075 commented May 29, 2017

@ymartin59: Thanks for the toolchains. Tvheadend 4.2.2 (PR #2763) compiles with all of them, except for hi3535, which fails building the ffmpeg-3.2.4 dependency. If I'm not mistaken the only model using this architecture is the NVR216, which is a dedicated surveillance system. Fix seems not straight forward. But before I spend a lot on time on this now, Do you think this actually needs fixing or can I safely "ignore" this architecture fo the TVH package in light of an NVR use-case?

@KaraokeStu
Copy link
Contributor

I just need confirmation from @Dr-Bean or @Diaoul that they are happy for me to merge these changes in, please hang-fire while I wait to hear from them (They've been a bit quiet lately - I imagine life has got in the way!)

@ymartin59
Copy link
Contributor Author

@KaraokeStu Thanks for the effort. These is still a question about how TC_FIRMWARE value should be set, either 6.1-15047-1 or 6.1-15047. What is your opinion about it ?

This was referenced Jun 5, 2017
@KaraokeStu
Copy link
Contributor

Hi @ymartin59 I'd drop the -1 off the end, I think that makes more sense and falls in line with all other versions

@ymartin59 ymartin59 force-pushed the dsm6-toolchains-6.1 branch from 975dbe1 to 4af4d2d Compare June 7, 2017 18:59
@ymartin59
Copy link
Contributor Author

I have replaced TC_FIRMWARE=6.1-15047

@ymartin59 ymartin59 force-pushed the dsm6-toolchains-6.1 branch from 4af4d2d to a7d4d89 Compare July 23, 2017 20:54
@ymartin59
Copy link
Contributor Author

@KaraokeStu I think it is now worth to merge this PR into dsm6 as it has been used to generate some packages by varia guys... by the way, it will ease package generation then.

@KaraokeStu KaraokeStu merged commit 248edb4 into SynoCommunity:dsm6 Jul 27, 2017
@ymartin59 ymartin59 deleted the dsm6-toolchains-6.1 branch September 19, 2017 19:19
jvega1976 pushed a commit to jvega1976/spksrc that referenced this pull request Nov 25, 2017
* 'dsm6' of https://github.com/jvega1976/spksrc: (83 commits)
  [DSM6] Add support for apollolake and denverton toolchains (6.1) (SynoCommunity#2942)
  [DSM 6] Add support for rtd1296 (SynoCommunity#2890)
  Update 6.1 toolchains digests
  Add dockerx64 DSM 6.1 toolchain
  DSM 6.1 toolchains (SynoCommunity#2727)
  Fix DSM 6.0.2 toolchains digests. SynoCommunity#2733 (SynoCommunity#2737)
  Radarr DSM6 Compatibility: var->.config + version checking fix (SynoCommunity#2746)
  Update Sonarr for DSM6 Compatibility: var->.config + version checking fix/improvement (SynoCommunity#2745)
  Borgbackup (SynoCommunity#2694)
  Determine correct user for web-based packages via BUILDNUMBER - Remove duplicate entries
  znc
  umurmur
  tvheadend
  tt-rss
  transmission
  syncthing
  subliminal
  squidguard
  sonarr
  sickrage
  ...
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.

5 participants