Page MenuHomeSolus
Differential D2084

Update to v0.14.48 & Include ufw rules
ClosedPublic
Actions

Authored by As4fN1v on Jan 26 2018, 11:26 AM.
Tags
None
Referenced Files
F11054770: D2084.id5025.diff
Fri, Aug 11, 1:10 AM
F11054769: D2084.id7842.diff
Fri, Aug 11, 1:10 AM
F11054768: D2084.id6611.diff
Fri, Aug 11, 1:10 AM
F11054767: D2084.id6142.diff
Fri, Aug 11, 1:10 AM
F11054766: D2084.id5026.diff
Fri, Aug 11, 1:10 AM
F11054765: D2084.id7762.diff
Fri, Aug 11, 1:10 AM
F11054764: D2084.id.diff
Fri, Aug 11, 1:10 AM
F11054763: D2084.id5603.diff
Fri, Aug 11, 1:10 AM
View All 40 Files
Subscribers
DataDrake
JoshStrobl
kyrios123
livingsilver94

Details

Reviewers
DataDrake
JoshStrobl
Group Reviewers
Triage Team
Commits
R2989:f90d0846fa8f: Update to v0.14.48 & Include ufw rules
Summary

Changelog

Signed-Off by: Asaf Niv <asafniv@disroot.org>

Test Plan

Use it for some time with 2+ GB of files.

Diff Detail

Repository
R2989 syncthing
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

As4fN1v created this revision.Jan 26 2018, 11:26 AM
Herald added a reviewer: Triage Team. · View Herald TranscriptJan 26 2018, 11:26 AM
As4fN1v updated this revision to Diff 5026.Jan 26 2018, 11:31 AM

Fixed pspec.

As4fN1v updated this revision to Diff 5603.Feb 21 2018, 5:27 PM

Update.

As4fN1v updated this revision to Diff 6142.Mar 14 2018, 12:02 PM
  • Update.
kyrios123 added a subscriber: kyrios123.Mar 14 2018, 2:03 PM

this must be rebased

As4fN1v added a comment.Mar 17 2018, 4:40 PM
In D2084#39058, @kyrios123 wrote:

this must be rebased

What do you mean?

livingsilver94 added a subscriber: livingsilver94.Mar 17 2018, 4:43 PM

He means you have to rework your patch since syncthing has already been updated: https://dev.solus-project.com/source/syncthing. Now you have to apply ufw rules only.

As4fN1v added a comment.Mar 31 2018, 4:51 PM
In D2084#39856, @livingsilver94 wrote:

He means you have to rework your patch since syncthing has already been updated: https://dev.solus-project.com/source/syncthing. Now you have to apply ufw rules only.

It's the same version as the Solus Repo.

DataDrake added a subscriber: DataDrake.Mar 31 2018, 5:26 PM
In D2084#41959, @As4fN1v wrote:
In D2084#39856, @livingsilver94 wrote:

He means you have to rework your patch since syncthing has already been updated: https://dev.solus-project.com/source/syncthing. Now you have to apply ufw rules only.

It's the same version as the Solus Repo.

Left hand side of the diff doesn't match the current state of the repo:

https://dev.solus-project.com/R2989:6551f3d5b75a8299a6e9ade858f80350c4a0be33

DataDrake requested changes to this revision.Mar 31 2018, 5:26 PM
This revision now requires changes to proceed.Mar 31 2018, 5:26 PM
As4fN1v updated this revision to Diff 6611.Apr 4 2018, 1:06 PM

Rebase.

As4fN1v updated this revision to Diff 6967.May 3 2018, 9:08 AM

Update to v0.14.47

As4fN1v retitled this revision from Include ufw rules to Update to v0.14.47 & Include ufw rules.May 3 2018, 9:09 AM
As4fN1v edited the summary of this revision. (Show Details)
As4fN1v updated this revision to Diff 6968.May 3 2018, 9:11 AM

Remove unnecessary space.

DataDrake requested changes to this revision.Jun 9 2018, 2:55 PM

Needs to be rebuild against updated synthing

This revision now requires changes to proceed.Jun 9 2018, 2:55 PM
As4fN1v updated this revision to Diff 7762.Jun 20 2018, 1:34 PM

Update to v0.14.48 & Include ufw rules

As4fN1v retitled this revision from Update to v0.14.47 & Include ufw rules to Update to v0.14.48 & Include ufw rules.Jun 20 2018, 1:35 PM
As4fN1v edited the summary of this revision. (Show Details)
As4fN1v edited the summary of this revision. (Show Details)
As4fN1v updated this revision to Diff 7763.Jun 20 2018, 1:37 PM
  • update email
As4fN1v added a comment.Jun 20 2018, 2:09 PM
In D2084#48320, @DataDrake wrote:

Needs to be rebuild against updated synthing

done

JoshStrobl requested changes to this revision.Jun 20 2018, 3:32 PM
JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
30–31

This isn't needed, you're using the install command already. Read what the -D flag does.

This revision now requires changes to proceed.Jun 20 2018, 3:32 PM
As4fN1v updated this revision to Diff 7842.Jun 25 2018, 12:34 PM
  • fixed
JoshStrobl requested changes to this revision.Jun 25 2018, 8:39 PM

Need to provide a test plan on how this was actually tested by you. Otheriwse LGTM.

This revision now requires changes to proceed.Jun 25 2018, 8:39 PM
As4fN1v requested review of this revision.Jun 26 2018, 12:41 PM
As4fN1v edited the test plan for this revision. (Show Details)
As4fN1v marked an inline comment as done.
In D2084#50463, @JoshStrobl wrote:

Need to provide a test plan on how this was actually tested by you. Otheriwse LGTM.

Fixed

JoshStrobl accepted this revision.Jul 11 2018, 9:46 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 11 2018, 9:48 PM
Closed by commit R2989:f90d0846fa8f: Update to v0.14.48 & Include ufw rules (authored by As4fN1v, committed by JoshStrobl). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
6 lines
147 lines

Diff 5025

View Options

package.yml

Loading...
View Options

pspec_x86_64.xml

Loading...
Copyright © 2015-2021 Solus Project. The Solus logo is Copyright © 2016-2021 Solus Project. All Rights Reserved.