Add Mobile-detect to packagist

[nMustaki]

Hello,

first congrats on your work !

I wanted to know if you could add mobile-detect to packagist, so your users can stay automatically up to date ?
It should not be a big deal of work, but I cannot offer a PR as I have no way to test it (I don't think you can delete a package once added to packagist.org or disown it to another personn). User Suncat2000 forked your repo a few months ago and created a package with this outdated version, which should be a base for creating your official package : https://github.com/suncat2000/php-mobile-detect/blob/master/composer.json

I have created a first draft in a gist : https://gist.github.com/4111592

Thanks

[serbanghita]

Thanks for this, it was suggested by other people as well. I don't use composer for the moment but I will find someone to test this!

[serbanghita]

The latest version from 'master' contains composer support.
Still have to do some reading at http://getcomposer.org/doc/04-schema.md and for the moment I cannot test composer because I'm behind a proxy and keep getting

[Composer\Downloader\TransportException]
The "https://packagist.org/packages.json" file could not be downloaded: failed to open stream: A socket operation was attempted to an unreachab

Will test it tonight. Thanks!

[suncat2000]

Hi
I tested the installation of class Mobile_Detect with composer.
Not working autoload of this class((
If you are using the old system of class naming, without namespaces, class name reflects the location of the class. When you get a class, symbol _ from class name replaced by / and tries to include file with class (If I'm not mistaken).

So if you want use PSR-0 autoload, you need change dir structure and rename class file:
Mobile_Detect.php
to
Mobile/Detect.php
The class name will remain the same (Mobile_Detect).

Or you can use classmap autoload, then you need change composer.json:

"autoload": {
        "psr-0": {
            "Mobile_Detect": ""
        }
    }

to

"autoload": {
        "classmap": ["Mobile_Detect.php"]
    }

[serbanghita]

@suncat2000 thanks a lot, testing it right now. I will stick with Mobile_Detect.php for the moment.

[serbanghita]

Done, it works now. See also https://packagist.org/packages/mobiledetect/mobiledetectlib , closing this.

Do you think I should include the /tests/ folder in the future releases?

[suncat2000]

Thanks, all good!
I think the tests will not interfere.

[nMustaki]

The tests folder won't be a problem, but maybe you need a debug package if you start using composer ?

Thanks anyways to both of you :-)

[serbanghita]

@nMustaki I want to create a debug package, can you elaborate on how I can do that with composer? I cannot find anything related to debugging in their docs.

[nMustaki]

Sorry it was misleading, I thought about using branches.
You should be able to create another branch on github with all debug
options you wish and create a package from it. Or have a devel git branch
from which you merge into master only the production code for stable
releases.

/*
** "What do you despise? By this you are truly known."
** from Manual of Muad'Dib by the Princess Irulan.
*/

On Mon, Nov 26, 2012 at 10:13 AM, Serban Ghita notifications@github.comwrote:

@nMustaki https://github.com/nMustaki I want to create a debug package,
can you elaborate on how I can do that with composer? I cannot find
anything related to debugging in their docs.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/18#issuecomment-10708785.

[serbanghita]

Agree, I'm currently discussing this here: #19