Skip to content

Commit

Permalink
#84 fix:autoloading while executing into phar
Browse files Browse the repository at this point in the history
  • Loading branch information
Niktux committed Nov 4, 2016
1 parent 27ccc0e commit 2730836
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion karma
100755 → 100644
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#!/usr/bin/env php
<?php

require 'vendor/autoload.php';
require __DIR__ . '/vendor/autoload.php';

$console = new Karma\Console(new Karma\Application());
$console->run();

5 comments on commit 2730836

@JBRulien44
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bonjour,
La constante DIR me génère une fatal error
php cherche un dossier vendor dans /vendor/niktux/karma
Est-ce dû à ma configuration ?
Merci d'avance

@Niktux
Copy link
Owner Author

@Niktux Niktux commented on 2730836 Nov 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bonjour,
C'est une régression effectivement.

La solution la plus rapide pour corriger est d'utiliser le phar. Le fait d'utiliser composer pour récupérer Karma est une erreur que j'avais identifié depuis longtemps : il peut y avoir un conflit entre les dépendances que j'utilise pour développer Karma et les dépendances de votre projet. Ce qui n'a aucun sens en soi !
C'est pour ça que je viens (enfin) de mettre en place la création automatique d'un phar pour chaque nouvelle version de Karma.

Pour corriger immédiatement votre erreur :

En parallèle je vais voir si je peux fixer le BC break

@Niktux
Copy link
Owner Author

@Niktux Niktux commented on 2730836 Nov 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si vous êtes pressés, la 5.5.1 corrige la régression.

Je vous encourage quand même à migrer vers l'utilisation du phar.

@JBRulien44
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci Niktux, Au faite Super boulot !!

@Niktux
Copy link
Owner Author

@Niktux Niktux commented on 2730836 Nov 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci, ça fait plaisir !

Please sign in to comment.