Uploaded image for project: 'phpBB3'
  1. phpBB3
  2. PHPBB3-11163

Include ext/ directory in installation and update packages

    Details

    • Type: Improvement
    • Status: Unverified Fix
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 3.1.0-dev
    • Fix Version/s: 3.1.0-b4
    • Component/s: None
    • Labels:
      None

      Description

      RFC: https://area51.phpbb.com/phpBB/viewtopic.php?f=108&t=43533

      The ext/ dir should be created when composer dependencies are installed. This way it does not have to be done manually. This can be done like this http://getcomposer.org/doc/articles/scripts.md using the post-install-cmd event by running the mkdir command.

      We decided to just add ext/index.htm to the repository.

        Issue Links

          Activity

          Hide
          imkingdavid David King added a comment -

          Instead of including the ext/ dir in the repo, we are going to use composer to create it; PHPBB3-11161 was for the former, this is for the latter, so I am closing it in favor of this.

          Show
          imkingdavid David King added a comment - Instead of including the ext/ dir in the repo, we are going to use composer to create it; PHPBB3-11161 was for the former, this is for the latter, so I am closing it in favor of this.
          Show
          Oleg Oleg [X] (Inactive) added a comment - Per http://area51.phpbb.com/phpBB/viewtopic.php?p=246969&sid=4526baf1d9c0256dfafa66b89286011d#p246969 the directory is going to be empty.
          Hide
          nickvergessen Joas Schilling added a comment -

          I would prefer putting it into the build script.
          Opinions?

          Show
          nickvergessen Joas Schilling added a comment - I would prefer putting it into the build script. Opinions?
          Hide
          VSE Matt Friedman added a comment -

          Isn't it time this gets done?

          Show
          VSE Matt Friedman added a comment - Isn't it time this gets done?
          Hide
          bantu Andreas Fischer added a comment -

          Yes

          Show
          bantu Andreas Fischer added a comment - Yes

            People

            • Assignee:
              nickvergessen Joas Schilling
              Reporter:
              imkingdavid David King
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development