Coder Social home page Coder Social logo

Comments (15)

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: DASPRiD on 03/29/12)

This looks kinda old. Evan, didn't we switch to \GlobIterator for that reason?

By the way, Yonni, did you test with the current beta3 or an old version?

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: yonman on 03/29/12)

Forked SkeletonApplication and Zf2 and sent pull requests:
zendframework/ZendSkeletonApplication#49
#980

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: yonman on 03/29/12)

Tested with beta3, found a TODO in ConfigListener so I took it up with Matthew. According to him globiterators were inapplicable, so I applied a simpler solution.

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: EvanDotPro on 03/29/12)

I had discussed this with Matthew and others on IRC. We discovered that \GlobIterator didn't solve the issue like we had anticipated it would (it doesn't support GLOB_BRACE at all, actually).

IIRC, Matthew was thinking GLOB_BRACE wouldn't work on Windows, but it turns out it does. It seems to only be non-GNU systems like Solaris that don't support it and I think the consensus was that people on those systems can easily change the glob pattern (and we could document such). Though, I'm unsure if passing GLOB_BRACE as a flag throws a warning or anything on non-GNU systems. Anyway, I suppose I don't mind changing this to two globs though simply to maintain a higher level of compatibility if there's no objections.

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: DASPRiD on 03/29/12)

I could come up with a fallback solution so GLOB_BRACE can still be used, even on non-GNU systems.

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: yonman on 04/01/12)

One more problematic point:
AIX glob calls seem to ignore cwd - I can only get correct results when we use an absolute path.
When testing, I verified with getcwd that we are in the document root, etc...

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: EvanDotPro on 04/13/12)

I'm not really sure there's much we can do to fix the AIX glob cwd issue...

It would seem to me that there are three options in this case:

  1. Simply adjust the glob paths to be absolute after cloning the skeleton.
  2. Use Zend\Module\Listener\ConfigListener::addConfigStaticPaths() to add each config file explicitly instead of addConfigGlobPaths() to add glob patterns.
  3. Re-implement glob() completely in PHP, which I'd prefer we not do.

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: EvanDotPro on 05/31/12)

This issue is resolved via [PR 983|https://github.com//pull/983] and [PR 1419|https://github.com//pull/1419].

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: yonman on 06/24/12)

The fix in PR 983 seem to work when I tested it in our IBMi machine but I had to create absolute paths for it to work. It seems that AIX glob works only with static paths - otherwise you get empty results and an empty configuration.

Should this issue be reopened for this? or should I create a new issue?

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

(Originally posted by: EvanDotPro on 06/28/12)

I'm re-opening and assigning to Ben Scholzen. Perhaps there is something he can add to our Zend\Stdlib\Glob class to account for this.

from zendframework.

zfbot avatar zfbot commented on June 12, 2024

This issue was ported from the ZF2 Jira Issue Tracker at
http://framework.zend.com/issues/browse/ZF2-243

Known GitHub users mentioned in the original message or comment:
@YonmaN, @DASPRiD, @EvanDotPro

from zendframework.

ralphschindler avatar ralphschindler commented on June 12, 2024

@EvanDotPro can we close this?

from zendframework.

DASPRiD avatar DASPRiD commented on June 12, 2024

No closing yet, we are still about to fix the relative-path issue on AIX.

from zendframework.

ThaDafinser avatar ThaDafinser commented on June 12, 2024

@DASPRiD no issue anymore?

from zendframework.

weierophinney avatar weierophinney commented on June 12, 2024

IIRC, we fixed this for 2.3.

from zendframework.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.