[MacPorts] #42596: new port: MobileDevice 1.0.100
#42596: new port: MobileDevice 1.0.100 --------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Keywords: | Port: --------------------------------+-------------------------------- A python package, and command line tool, which wraps Apple's MobileDevice API - providing access to iOS devices. -- Ticket URL: <https://trac.macports.org/ticket/42596> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: | ---------------------------------+-------------------------------- Comment (by cal@…): You don't need to set `name` since `github.setup` already does that for you. How did you come up with the version number? It doesn't seem upstream is providing any version numbering (or tags, for that matter), so our course of action usually is the following: - Ask upstream to provide versioning and tags in their git-repo for us to track. - If upstream isn't willing to do that, reference a commit (like you did) and use a date in the form YYYYMMDD as version number. The port fails lint (`port lint --nitpick`), because it doesn't set a `license` field (effectively preventing binary re-distribution). https://github.com/mountainstorm/MobileDevice/blob/master/LICENSE suggests the project is MIT-licensed, so `license MIT` would be appropriate. Note that the MIT license requires us to ship the license file: {{{ The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. }}} So you'd have to add something along the lines of: {{{ post-destroot { xinstall -m0755 -d ${destroot}${prefix}/share/doc/${name} xinstall -m0644 ${worksrcpath}/LICENSE ${destroot}${prefix}/share/doc/${name}/ } }}} -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: | ---------------------------------+-------------------------------- Changes (by cal@…): * cc: cal@… (added) -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: | ---------------------------------+-------------------------------- Comment (by cal@…): You might also want to ask upstream to incorporate installing the license file into their installation routine. -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: | ---------------------------------+-------------------------------- Comment (by schmurtz_macport@…):
How did you come up with the version number?
Upstream is using the "1.0." + (commit count) as a version number, as implemented in https://github.com/mountainstorm/MobileDevice/blob/master/setup.py: {{{ version=u'1.0' + getcommit(), }}} \\
The port fails lint (port lint --nitpick), because it doesn't set a license field (effectively preventing binary re-distribution).
Portfile updated -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: Port: | ---------------------------------+-------------------------------- Comment (by cal@…): This port currently installs a file at `$prefix/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7 /site-packages/Distutils-1.0-py2.7.egg-info`. I've reported and I'd like to wait for https://github.com/mountainstorm/MobileDevice/issues/16 to be fixed before committing this to avoid conflicts with the distutils python package. Also reported - https://github.com/mountainstorm/MobileDevice/issues/17 to request tags and - https://github.com/mountainstorm/MobileDevice/issues/18 to request installing the license file. -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: Port: MobileDevice | ---------------------------------+-------------------------------- Changes (by macsforever2000@…): * version: 2.2.1 => * type: defect => submission * port: => MobileDevice -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: Port: MobileDevice | ---------------------------------+-------------------------------- Comment (by ryandesign@…): Why have you set distname? The github portgroup should work with any distname (when fetching from tags or commits), so there's no reason to change it from its default, and changing it to a value that does not include a version number or other similar identifier is problematic and requires [wiki:PortfileRecipes#unversioned-distfiles additional steps be taken]. -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#42596: new port: MobileDevice 1.0.100 ---------------------------------+-------------------------------- Reporter: schmurtz_macport@… | Owner: macports-tickets@… Type: submission | Status: closed Priority: Normal | Milestone: Component: ports | Version: Resolution: fixed | Keywords: Port: MobileDevice | ---------------------------------+-------------------------------- Changes (by cal@…): * status: new => closed * resolution: => fixed Comment: It seems upstream isn't going to respond… in that case: committed in r117783 with the following changes: - Applied a patch that fixes the package name issue - Fixed the package version number (because it can't be determined from git in our setup) - Avoid setting distname by renaming the worksrcdir to "MobileDevice" after extract, since the build system expects that Thank you for your submission! -- Ticket URL: <https://trac.macports.org/ticket/42596#comment:8> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts