Opened 2 years ago

Closed 2 years ago

#49032 closed submission (fixed)

git-cvs @20150817: new submission

Reported by: jpo@… Owned by: mojca (Mojca Miklavec)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: git-cvs

Description

First time porting python stuff, so I probably did something wrong.

Depends on rcsparse (just submitted in #49030).

Attachments (1)

Portfile (859 bytes) - added by jpo@… 2 years ago.

Download all attachments as: .zip

Change History (5)

Changed 2 years ago by jpo@…

comment:1 Changed 2 years ago by mojca (Mojca Miklavec)

  • Owner changed from macports-tickets@… to mojca@…
  • Version 2.3.3 deleted

comment:2 Changed 2 years ago by mojca (Mojca Miklavec)

I committed the port under r143004.

  • I added the missing build dependency on py-setuptools (otherwise installation under the the trace would fail)
  • I added the missing runtime dependency on py-rcsparse as you suggested
  • I took the latest release (if there is any reason against doing that or if you would want the latest commit, let me know)
  • I had to patch setup.py, otherwise the port would install files under /usr/local/libexec which is forbidden, undesired and more or less useless (one should probably report a bug upstream)

comment:3 Changed 2 years ago by mojca (Mojca Miklavec)

I also had to add a dependency on py-nose in r143005 to avoid a build failure on the buildbots. I wonder if this is an error in setup.py because the dependency is not really needed unless one also runs all the tests. And as I had to modify the Portfile anyway, I also added test.run yes to enable testing the module with "sudo port test git-cvs".

comment:4 Changed 2 years ago by mojca (Mojca Miklavec)

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.