search

propelorm / Propel

Current stable (and outdated and unmaintained) version of Propel - Please use v2 https://github.com/propelorm/Propel2 --
http://www.propelorm.org
MIT License
841 stars 417 forks source link

propel-gen output: Buildfile: /build.xml does not exist! #388

Closed meglio closed 12 years ago

meglio commented 12 years ago

Entry in composer.json: "propel/propel1": "1.6.*" Entry in composer.lock:

{ "package": "propel/propel1", "version": "dev-master", "source-reference": "da1e6ce1097cccd683890c301e0f39c5a9c988a0" }

~/public_html/db$ ../vendor/propel/propel1/generator/bin/propel-gen Buildfile: /build.xml does not exist! ~/public_html/db$ ../vendor/bin/propel-gen Buildfile: /build.xml does not exist!

Ubuntu 12.04 LTS ... but works on my Win7.

willdurand commented 12 years ago

Is is still the case?

meglio commented 12 years ago

php composer.phar update did not update Propel; My entry in composer.json is: "propel/propel1": "1.6.*"

meglio commented 12 years ago

.... ok I've removed everything from vendor and just re-installed it again and it works now.

Current entry in composer.lock:

    {
        "package": "propel/propel1",
        "version": "1.6.5"
    },
willdurand commented 12 years ago

1.6.* should match 1.6.x-dev

meglio commented 12 years ago

In composer.json I have "propel/propel1": "1.6.*", and in lock file the strings above.

willdurand commented 12 years ago

what do you get by running composer update?

2012/6/25 Anton Andriyevskyy < reply@reply.github.com

In composer.json I have "propel/propel1": "1.6.*", and in lock file the strings above.

Reply to this email directly or view it on GitHub: https://github.com/propelorm/Propel/issues/388#issuecomment-6549125

meglio commented 12 years ago

Tried it right now, here is what I get:

>php composer.phar update
Updating dependencies
Nothing to install or update
Generating autoload files

>php composer.phar -version
Composer version adaa788

>cat composer.json | grep propel
"propel/propel1": "1.6.*",

and in composer.lock I have this entry:

{
  "package": "propel/propel1",
  "version": "1.6.5"
}
willdurand commented 12 years ago

Well, require dev-master explicitly hen.

meglio commented 12 years ago

must it say "1.6.dev-master" or just "dev-master"? Please advice. Composer documentation is not clear at this point.

willdurand commented 12 years ago

Just dev-master sir :)

2012/6/25 Anton Andriyevskyy < reply@reply.github.com

must it say "1.6.dev-master" or just "dev-master"? Please advice. Composer documentation is not clear at this point.

Reply to this email directly or view it on GitHub: https://github.com/propelorm/Propel/issues/388#issuecomment-6549748

meglio commented 12 years ago

Ok, will test now.

Anyhow, here is another strange issue (see last line):

  - Removing propel/propel1 (1.6.5)
  - Installing propel/propel1 (dev-master)
    Cloning 3fc74ccffb05931ec3187b0dcff77dce732ef325

Skipped installation of generator/bin/propel-gen.bat for package propel/propel1, name conflicts with an existing file
willdurand commented 12 years ago

Na, it's quite normal.

2012/6/25 Anton Andriyevskyy < reply@reply.github.com

Ok, will test now.

Anyhow, here is another strange issue (see last line):

 - Removing propel/propel1 (1.6.5)
 - Installing propel/propel1 (dev-master)
   Cloning 3fc74ccffb05931ec3187b0dcff77dce732ef325

Skipped installation of generator/bin/propel-gen.bat for package
propel/propel1, name conflicts with an existing file

Reply to this email directly or view it on GitHub: https://github.com/propelorm/Propel/issues/388#issuecomment-6549850

meglio commented 12 years ago
  1. Looks like it still uses global Phing installation, not the one from "vendor" folder.
  2. Buildfile: C:\git\build.xml does not exist is still there (see at the bottom).
D:\uniserver\vhosts\newsintel.meg\db>..\vendor\bin\propel-gen
PHP Strict standards:  Declaration of Win32FileSystem::compare() should be compatible with FileSystem::compare($f1, $f2) in D:\wamp\www\phin
g-2.4.6\classes\phing\system\io\Win32FileSystem.php on line 475
PHP Stack trace:
PHP   1. {main}() D:\wamp\www\phing-2.4.6\bin\phing.php:0
PHP   2. Phing::fire($args = *uninitialized*) D:\wamp\www\phing-2.4.6\bin\phing.php:37
PHP   3. Phing::start($args = *uninitialized*, $additionalUserProperties = *uninitialized*) D:\wamp\www\phing-2.4.6\classes\phing\Phing.php:
270
PHP   4. Phing->execute($args = *uninitialized*) D:\wamp\www\phing-2.4.6\classes\phing\Phing.php:157
PHP   5. PhingFile->__construct($arg1 = *uninitialized*, $arg2 = *uninitialized*) D:\wamp\www\phing-2.4.6\classes\phing\Phing.php:349
PHP   6. FileSystem::getFileSystem() D:\wamp\www\phing-2.4.6\classes\phing\system\io\PhingFile.php:56
PHP   7. include_once() D:\wamp\www\phing-2.4.6\classes\phing\system\io\FileSystem.php:88

Strict standards: Declaration of Win32FileSystem::compare() should be compatible with FileSystem::compare($f1, $f2) in D:\wamp\www\phing-2.4
.6\classes\phing\system\io\Win32FileSystem.php on line 475

Call Stack:
    0.0075     140752   1. {main}() D:\wamp\www\phing-2.4.6\bin\phing.php:0
    0.1473    2711856   2. Phing::fire(???) D:\wamp\www\phing-2.4.6\bin\phing.php:37
    0.1473    2711904   3. Phing::start(???, ???) D:\wamp\www\phing-2.4.6\classes\phing\Phing.php:270
    0.1474    2712392   4. Phing->execute(???) D:\wamp\www\phing-2.4.6\classes\phing\Phing.php:157
    0.1475    2714584   5. PhingFile->__construct(???, ???) D:\wamp\www\phing-2.4.6\classes\phing\Phing.php:349
    0.1475    2714744   6. FileSystem::getFileSystem() D:\wamp\www\phing-2.4.6\classes\phing\system\io\PhingFile.php:56
    0.1481    2716320   7. include_once('D:\wamp\www\phing-2.4.6\classes\phing\system\io\WinNTFileSystem.php') D:\wamp\www\phing-2.4.6\class
es\phing\system\io\FileSystem.php:88

Buildfile: C:\git\build.xml does not exist!
meglio commented 12 years ago

and in ubuntu where phing 2.4.12 is used:

~/public_html/db$ ../vendor/propel/propel1/generator/bin/propel-gen
Buildfile: /build.xml does not exist!
meglio commented 12 years ago

Still issue on ubuntu with phing 2.4.12 installed globaly and using dev-master for propel in composer.

willdurand commented 12 years ago

@meglio really? Even with @grEvenX's patch?

meglio commented 12 years ago

Is that patch already embedded in dev-master? I'm confused. I just used composer with dev-master reference, but not applied patch in any way - must I do it somehow to test?

grEvenX commented 12 years ago

@willdurand @meglio He is using dev-master so I guess he haven't gotten my changes (the pull-request hasn't been merged in)

meglio commented 12 years ago

Yes I think so. Right now:

composer.json

"propel/propel1": "dev-master",

composer.lock

{
    "package": "propel/propel1",
    "version": "dev-master",
    "source-reference": "bb34bdab83aa0f7e5a2fcf11dece63352efabbcf",
    "commit-date": "1341218897"
}
willdurand commented 12 years ago

Makes sense ;) @meglio could you try the patch?

2012/7/3 Even André Fiskvik < reply@reply.github.com

@willdurand @meglio He is using dev-master so I guess he haven't gotten my changes (the pull-request hasn't been merged in)

Reply to this email directly or view it on GitHub: https://github.com/propelorm/Propel/issues/388#issuecomment-6729748

meglio commented 12 years ago

Is there a way to try it with composer?

grEvenX commented 12 years ago

@meglio not easily done, no.

This is servering you with a silver-spoon, but here you go :)

Download this file and unpack it: https://gist.github.com/gists/7512e19cf085c346280e/download

Move the patchfile (388-fix-propel-gen-path.patch) in the unpacked folder to your composer/vendor/propel/propel1 directory. Cd to that directory and enter: patch -p0 < 388-fix-propel-gen-path.patch

meglio commented 12 years ago

Ok, after patching it works:

> patch -p0 < 388-fix-propel-gen-path.patch
patching file generator/bin/phing.php
patching file generator/bin/propel-gen
patch unexpectedly ends in middle of line
Hunk #1 succeeded at 18 with fuzz 1.

> ../../vendor/propel/propel1/generator/bin/propel-gen -q

BUILD FINISHED

Total time: 0.3491 seconds

Will it be merged soon?

willdurand commented 12 years ago

sure, now

meglio commented 12 years ago

Excellent and thanks, it works now.

davidloubere commented 12 years ago

Hi, that patch seems not to be working with Ubuntu 11.04 :

patching file generator/bin/phing.php Hunk #1 FAILED at 12. 1 out of 1 hunk FAILED -- saving rejects to file generator/bin/phing.php.rej patching file generator/bin/propel-gen patch unexpectedly ends in middle of line Hunk #1 FAILED at 18. 1 out of 1 hunk FAILED -- saving rejects to file generator/bin/propel-gen.rej

grEvenX commented 12 years ago

@davidloubere no reason to use the patch anymore, it's already merged in master branch...

davidloubere commented 12 years ago

It works on the master branch indeed. Thanks you.