New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #15975 (closed update: fixed)

Opened 5 years ago

Last modified 4 years ago

git-core update to 1.5.6.3

Reported by: maccheck@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 1.6.0
Keywords: git-core Portfile patch Cc: bryan@…, raimue@…
Port:

Description

version bumping git-core from 1.5.6.2 to 1.5.6.3, again ;-)

Attachments

Portfile_1.5.6.2_to_1.5.6.3.patch (1.5 KB) - added by maccheck@… 5 years ago.

Change History

Changed 5 years ago by maccheck@…

comment:1 Changed 5 years ago by raimue@…

maccheck, as you are continuously providing updates for git-core, I would like to add you to the list of maintainers of git-core.

Bryan, do you have any objections?

comment:2 Changed 5 years ago by bryan@…

No objections, as long as you guys keep running the test suite and a sanity test before checking in! Things have been running smoothly for the last while, so I'm content with how you're running things.

comment:3 Changed 5 years ago by ryandesign@…

  • Cc raimue@… added

comment:4 follow-up: ↓ 5 Changed 5 years ago by raimue@…

The test suite is kind of useless for git-core in MacPorts, because it always fails when using root privileges.

*** t0004-unwritable.sh ***
*   ok 1: setup
* FAIL 2: write-tree should notice unwritable repository


                (
                        chmod a-w .git/objects
                        test_must_fail git write-tree
                )
                status=$?
                chmod 775 .git/objects
                (exit $status)

root always has permission to write, even when chmod a-w is done before. This will become better when the GSoC project for dropping privileges is completed. But until then, I don't see an easy way to run the test suite and rely on it.

Workaround:

cd $(port work git-core)/git-$(port -q info --version git-core)
sudo chown -R $USER .
make test

comment:5 in reply to: ↑ 4 Changed 5 years ago by maccheck@…

I saw on the Git mailing list that there is already a patch for this test, so that it will be skipped if run with root privileges. I don't know if this will get accepted or not, nevertheless.

http://kerneltrap.org/mailarchive/git/2008/6/30/2296944

comment:6 Changed 5 years ago by raimue@…

  • Status changed from new to closed
  • Resolution set to fixed

Committed in r38355.

Added maccheck to the list of maintainers in r38356.

comment:7 Changed 4 years ago by jmr@…

  • Type changed from enhancement to update

comment:8 Changed 4 years ago by anonymous

  • Milestone Port Updates deleted

Milestone Port Updates deleted

Note: See TracTickets for help on using tickets.