Странное поведение скриптового хука. Commit-mime-type-check.pl - PullRequest
0 голосов
/ 21 ноября 2011

Я скачал сценарий перехвата pre-commit "commit-mime-type-check.pl" из svn.apache.org Я работаю над окнами, есть только две вещи, чтобы изменить путь к двоичному файлу svnlookи путь к временному каталогу. Я сделал это.но я получаю следующую ошибку

Error: Commit failed (details follow):  
Error: Commit blocked by pre-commit hook (exit code 1) with output:  
Error: '-' is not recognized as an internal or external command,  
Error: operable program or batch file.  
Error: C:\vsvn\Repositories\myproj\hooks\local_commit_mime_eol_check.pl:   
Error: `C:\vsvn\bin\svnlook.exe changed C:\vsvn\Repositories\myproj -t 16-1l' failed   
Error: with no output.  
Finished!: 

Я отладил самостоятельно и обнаружил, что происходит сбой.

my $pid = open(SAFE_READ, '-|');

Я прочитал объяснение об открытии с "- |"на [_ http://perldoc.perl.org/perlipc.html#Using-open()-for-IPC] под безопасным каналом Открыть

Безопасный канал открывается Еще один интересный подход к IPC - заставить вашу единую программу работать в многопроцессорном режиме и обмениваться данными между собой или даже между собой.Функция open () будет принимать аргумент файла либо "- |"или "| -", чтобы сделать очень интересную вещь: он разветвляет ребенка, подключенного к файловому дескриптору, который вы открыли.Ребенок выполняет ту же программу, что и родитель.Это полезно для безопасного открытия файла, например, при работе с предполагаемым UID или GID.Если вы открываете канал в минус, вы можете записать в файловый дескриптор, который вы открыли, и ваш ребенок найдет его в своем STDIN.Если вы открываете канал из минуса, вы можете прочитать из файлового дескриптора, который вы открыли, все, что ваш ребенок пишет в свой STDOUT.

То, что я не могу понять, это из-за несовместимостиили что.Как он изначально был написан на linux box, он очень богат для связи и межпроцессного взаимодействия.где я всегда нахожу окно плохим. (Никому не обижай. Это мое личное чувство. Давайте вернемся к теме. Я ищу все дикие предложения. Не ищу исправлений, но если я получу, я был бы признателен

Я попробовал все, но не смог, наконец, я попытался гуглить по этому поводу, если кто-то также сталкивается с той же проблемой. Не много ссылок, но нашел этот и интересная часть это, этоне решен. Тогда у меня нет возможности, кроме как спросить здесь.

Ниже приведен код сценария для вашей справки

#!/usr/bin/env perl

# ====================================================================
# commit-mime-type-check.pl: check that every added file has the
# svn:mime-type property set and every added file with a mime-type
# matching text/* also has svn:eol-style set. If any file fails this
# test the user is sent a verbose error message suggesting solutions and
# the commit is aborted.
#
# Usage: commit-mime-type-check.pl REPOS TXN-NAME
# ====================================================================
# Most of commit-mime-type-check.pl was taken from
# commit-access-control.pl, Revision 9986, 2004-06-14 16:29:22 -0400.
# ====================================================================
# Copyright (c) 2000-2004 CollabNet.  All rights reserved.
#
# This software is licensed as described in the file COPYING, which
# you should have received as part of this distribution.  The terms
# are also available at http://subversion.tigris.org/license.html.
# If newer versions of this license are posted there, you may use a
# newer version instead, at your option.
#
# This software consists of voluntary contributions made by many
# individuals.  For exact contribution history, see the revision
# history and logs, available at http://subversion.tigris.org/.
# ====================================================================

# Turn on warnings the best way depending on the Perl version.
BEGIN {
  if ( $] >= 5.006_000)
    { require warnings; import warnings; }
  else
    { $^W = 1; }
}

use strict;
use Carp;


######################################################################
# Configuration section.

# Svnlook path.
my $svnlook = "/usr/bin/svnlook";

# Since the path to svnlook depends upon the local installation
# preferences, check that the required program exists to insure that
# the administrator has set up the script properly.
{
  my $ok = 1;
  foreach my $program ($svnlook)
    {
      if (-e $program)
        {
          unless (-x $program)
            {
              warn "$0: required program `$program' is not executable, ",
                   "edit $0.\n";
              $ok = 0;
            }
        }
      else
        {
          warn "$0: required program `$program' does not exist, edit $0.\n";
          $ok = 0;
        }
    }
  exit 1 unless $ok;
}

######################################################################
# Initial setup/command-line handling.

&usage unless @ARGV == 2;

my $repos        = shift;
my $txn          = shift;

unless (-e $repos)
  {
    &usage("$0: repository directory `$repos' does not exist.");
  }
unless (-d $repos)
  {
    &usage("$0: repository directory `$repos' is not a directory.");
  }

# Define two constant subroutines to stand for read-only or read-write
# access to the repository.
sub ACCESS_READ_ONLY  () { 'read-only' }
sub ACCESS_READ_WRITE () { 'read-write' }


######################################################################
# Harvest data using svnlook.

# Change into /tmp so that svnlook diff can create its .svnlook
# directory.
my $tmp_dir = '/tmp';
chdir($tmp_dir)
  or die "$0: cannot chdir `$tmp_dir': $!\n";

# Figure out what files have added using svnlook.
my @files_added;
foreach my $line (&read_from_process($svnlook, 'changed', $repos, '-t', $txn))
  {
        # Add only files that were added to @files_added
    if ($line =~ /^A.  (.*[^\/])$/)
      {
        push(@files_added, $1);
      }
  }

my @errors;
foreach my $path ( @files_added )
    {
        my $mime_type;
        my $eol_style;

        # Parse the complete list of property values of the file $path to extract
        # the mime-type and eol-style
        foreach my $prop (&read_from_process($svnlook, 'proplist', $repos, '-t',
                          $txn, '--verbose', $path))
            {
                if ($prop =~ /^\s*svn:mime-type : (\S+)/)
                    {
                        $mime_type = $1;
                    }
                elsif ($prop =~ /^\s*svn:eol-style : (\S+)/)
                    {
                        $eol_style = $1;
                    }
            }

        # Detect error conditions and add them to @errors
        if (not $mime_type)
            {
                push @errors, "$path : svn:mime-type is not set";
            }
        elsif ($mime_type =~ /^text\// and not $eol_style)
            {
                push @errors, "$path : svn:mime-type=$mime_type but svn:eol-style is not set";
            }
    }

# If there are any errors list the problem files and give information
# on how to avoid the problem. Hopefully people will set up auto-props
# and will not see this verbose message more than once.
if (@errors)
  {
    warn "$0:\n\n",
         join("\n", @errors), "\n\n",
                 <<EOS;

    Every added file must have the svn:mime-type property set. In
    addition text files must have the svn:eol-style property set.

    For binary files try running
    svn propset svn:mime-type application/octet-stream path/of/file

    For text files try
    svn propset svn:mime-type text/plain path/of/file
    svn propset svn:eol-style native path/of/file

    You may want to consider uncommenting the auto-props section
    in your ~/.subversion/config file. Read the Subversion book
    (http://svnbook.red-bean.com/), Chapter 7, Properties section,
    Automatic Property Setting subsection for more help.
EOS
    exit 1;
  }
else
  {
    exit 0;
  }

sub usage
{
  warn "@_\n" if @_;
  die "usage: $0 REPOS TXN-NAME\n";
}

sub safe_read_from_pipe
{
  unless (@_)
    {
      croak "$0: safe_read_from_pipe passed no arguments.\n";
    }
  print "Running @_\n";
  my $pid = open(SAFE_READ, '-|');
  unless (defined $pid)
    {
      die "$0: cannot fork: $!\n";
    }
  unless ($pid)
    {
      open(STDERR, ">&STDOUT")
        or die "$0: cannot dup STDOUT: $!\n";
      exec(@_)
        or die "$0: cannot exec `@_': $!\n";
    }
  my @output;
  while (<SAFE_READ>)
    {
      chomp;
      push(@output, $_);
    }
  close(SAFE_READ);
  my $result = $?;
  my $exit   = $result >> 8;
  my $signal = $result & 127;
  my $cd     = $result & 128 ? "with core dump" : "";
  if ($signal or $cd)
    {
      warn "$0: pipe from `@_' failed $cd: exit=$exit signal=$signal\n";
    }
  if (wantarray)
    {
      return ($result, @output);
    }
  else
    {
      return $result;
    }
}

sub read_from_process
  {
  unless (@_)
    {
      croak "$0: read_from_process passed no arguments.\n";
    }
  my ($status, @output) = &safe_read_from_pipe(@_);
  if ($status)
    {
      if (@output)
        {
          die "$0: `@_' failed with this output:\n", join("\n", @output), "\n";
        }
      else
        {
          die "$0: `@_' failed with no output.\n";
        }
    }
  else
    {
      return @output;
    }
}

1 Ответ

0 голосов
/ 21 ноября 2011

Я продолжаю смотреть на этот синтаксис Perl и говорю себе: " Ух, это старый ".

Если вы посмотрите на Perldoc функции open , вы увидите, что аргумент с двумя параметрами не упоминает, что делает ваш скрипт-ловушка.Тем не менее, как вы указали, Perlipc упоминает об этом.

Похоже, ваш интерпретатор Perl интерпретирует |-, как упомянуто в описании открытых функций Perldoc:

  • Это двухпараметрическая форма функции.Таким образом:
    • Первый аргумент - это имя дескриптора файла.
    • Вторым аргументом является имя файла.
  • Поскольку имя файла заканчивается каналом, остальная часть имени файла - это имя команды длявыполнить, что для передачи через STDIN, когда вы читаете из канала.

Поскольку второй аргумент - -|, он ищет команду с именем -.Нет такой команды с именем -, функция open не работает.

Не уверен, что вы можете сделать, но вы можете позаимствовать мой Pre-commit Kitchen Sink Hook , который может сделать целоемножество вещей.

Одна из вещей, которые он может сделать, - это принудительно устанавливать типы свойств и значения на основе имен файлов.Например, вы можете заставить все файлы, которые заканчиваются суффиксом .sh, иметь svn:eol-type равным LF, или что файлы с суффиксом .pdf имеют svn:mime-type, установленным на application/pdf, что, как я считаю,что должен делать этот хук.

Я еще немного расскажу об этом хук-скрипте, чтобы посмотреть, смогу ли я точно выяснить, что он делает и как он работает.Я отправил бы исправленную версию сценария подключения в Apache, но они больше не поддерживают сценарии подключения.

Я никогда раньше не использовал синтаксис open SAFE_READ, "-|", но вы чему-то учитесь каждый день.Мне придется поиграть с этим.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...