Page MenuHomeFreeBSD

Update devel/gitaly to 0.82.0

Authored by mfechner on Mar 27 2018, 11:32 AM.



Commit message:
Update to version 0.82.0 which is required for gitlab 10.5.x.

Reviewed by: tz/swills (mentor)
Approved by: tz/swills (mentor)
Differential Revision: URL to this revision

Test Plan

All patches are build, you can find all build logs:

The fresh packages are tested against a new installation using:

An old version is updated following the manual:

Diff Detail

rP FreeBSD ports repository
Lint Skipped
Unit Tests Skipped

Event Timeline

Could you use devel/arcanist, or at least generate a diff with full context like it does, with svn diff -x -U9999 or git diff -U9999.



If you would like to see a full diff, you can also check it my gitlab installation:


could we please postpone this to the next release. This is a security release which was very time consuming to test and we should bring it as fast as possible to the repository.
I will address such changes later, as it is only cosmetic nature.

Is this ok for you?

mfechner added inline comments.

It is here tracked to not forget it:

In D14867#312863, wrote:

If you would like to see a full diff, you can also check it my gitlab installation:

I would like you to upload the full diff here, in the FreeBSD code review tool, not in some other tool that I'll have to understand how it works and all.

@mat I attached you the diff generated with svn diff -x -U9999

mfechner added inline comments.

Dear @mat

I tried now to use USE_GITLAB but it seems not to work.
I see the error message:

It looks to me that FreeBSD does not handle the directories correctly.

I will attach you the full diff, maybe you can help here.

tz requested changes to this revision.Apr 3 2018, 9:23 AM
tz added inline comments.

There is an whitespace missing

17 ↗(On Diff #41014)

I highly doubt the use of "bundle install" since it will very likely install something and ignores pkg. This defeats the purpose of the dependencies and should be removed. Or did i miss something?

This revision now requires changes to proceed.Apr 3 2018, 9:23 AM

You forgot to remove the post-extract target.

Also, you should probably change all occurrences of WRKDIR/DISTNAME to WRKSRC.

Side note, USE_GITLAB and all associated variables must happen in the same block as USES.

mfechner added inline comments.
17 ↗(On Diff #41014)

Dear @tz yes you miss something.
The procedure it to delete the Gemfile.lock to not depend on the exact version the Gemfile.lock mentions and then run bundle install --local to generate a new Gemfile.lock which only includes the local installed gems. No installation is happening as --local is used.

mfechner marked an inline comment as done.

Updated diff from comments.


Thanks a lot @mat I fixed it and testbuild was fine.

This revision is now accepted and ready to land.Apr 5 2018, 4:13 PM
This revision was automatically updated to reflect the committed changes.